Adblock setting should also set ads

Auditors: @emerick Fix brave/brave-browser#1317
brave · Sep 29, 2018 · cea3513 · cea3513
1 parent 28e1ef0
commit cea3513
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/browser/ui/webui/settings/default_brave_shields_handler.cc b/browser/ui/webui/settings/default_brave_shields_handler.cc
@@ -79,6 +79,13 @@ void DefaultBraveShieldsHandler::SetAdControlType(const base::ListValue* args) {
         CONTENT_SETTINGS_TYPE_PLUGINS,
         brave_shields::kAds,
         value == "allow" ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_BLOCK);
+  HostContentSettingsMapFactory::GetForProfile(profile_)->
+      SetContentSettingCustomScope(
+        ContentSettingsPattern::Wildcard(),
+        ContentSettingsPattern::Wildcard(),
+        CONTENT_SETTINGS_TYPE_PLUGINS,
+        brave_shields::kTrackers,
+        value == "allow" ? CONTENT_SETTING_ALLOW : CONTENT_SETTING_BLOCK);
 }
 
 void DefaultBraveShieldsHandler::GetCookieControlType(const base::ListValue* args) {