Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH-2283/Ad-blocker-off-by-default-in-Dawn #693

Closed
wants to merge 10 commits into from

Move disabling of ad block in Dawn to guard against race condition

  • Loading branch information
wlycdgr committed Feb 19, 2021
commit bcc2cf787823628a03b8a95188bf218927673395
@@ -1548,6 +1548,9 @@ function initializeGhosteryModules() {
// Make sure that getBrowserInfo() has resolved before we set these properties
(async() => {
await globals.BROWSER_INFO_READY;
if (globals.JUST_INSTALLED && BROWSER_INFO.name === 'ghostery_desktop') { // i.e., Dawn
conf.enable_ad_block = false; // GH-2283
}
if (IS_FIREFOX && BROWSER_INFO.name !== 'ghostery_desktop' && BROWSER_INFO.name !== 'ghostery_android') {
if (globals.JUST_INSTALLED) {
conf.enable_human_web = false;
@@ -1636,7 +1639,6 @@ function initializeGhosteryModules() {
(async() => {
await globals.BROWSER_INFO_READY;
if (BROWSER_INFO.name === 'ghostery_desktop') { // i.e., Dawn
conf.enable_ad_block = false; // GH-2283
chrome.tabs.create({
url: chrome.runtime.getURL('./app/templates/dawn_hub.html?justInstalled=true'),
active: true
ProTip! Use n and p to navigate between commits in a pull request.