Issue 3841 - Remove workaround stripping [style] selectors on certain websites

background.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 25 matching lines @@
 var FilterNotifier = require("filterNotifier").FilterNotifier;
 var SpecialSubscription = require("subscriptionClasses").SpecialSubscription;
 var ElemHide = require("elemHide").ElemHide;
 var defaultMatcher = require("matcher").defaultMatcher;
 var Prefs = require("prefs").Prefs;
 var updateIcon = require("icon").updateIcon;
 var showNextNotificationForUrl = require("notificationHelper").showNextNotificationForUrl;
 var port = require("messaging").port;
 var devtools = require("devtools");
 
-// Special-case domains for which we cannot use style-based hiding rules.
-// See http://crbug.com/68705.
-var noStyleRulesHosts = ["mail.google.com", "mail.yahoo.com", "www.google.com"];
-
 var htmlPages = new ext.PageMap();
 
 var contextMenuItem = {
   title: ext.i18n.getMessage("block_element"),
   contexts: ["image", "video", "audio"],
   onclick: function(page)
   {
     page.sendMessage({type: "composer.content.contextMenuClicked"});
   }
 };
@@ skipping 49 matching lines @@
       else
         filters.push(filter.text);
     }
   }
 
   return {filters: filters, exceptions: exceptions};
 }
 
 port.on("get-selectors", function(msg, sender)
 {
-  var selectors = [];
+  var selectors;
   var trace = devtools && devtools.hasPanel(sender.page);
 
   if (!checkWhitelisted(sender.page, sender.frame,
                         RegExpFilter.typeMap.DOCUMENT |
                         RegExpFilter.typeMap.ELEMHIDE))
-  {
-    var noStyleRules = false;
-    var specificOnly = checkWhitelisted(sender.page, sender.frame,
-                                        RegExpFilter.typeMap.GENERICHIDE);
-    var host = extractHostFromFrame(sender.frame);
-
-    for (var i = 0; i < noStyleRulesHosts.length; i++)
-    {
-      var noStyleHost = noStyleRulesHosts[i];
-      if (host == noStyleHost || (host.length > noStyleHost.length &&
-                                  host.substr(host.length - noStyleHost.length - 1) == "." + noStyleHost))
-      {
-        noStyleRules = true;
-      }
-    }
-    selectors = ElemHide.getSelectorsForDomain(host, specificOnly);
-    if (noStyleRules)
-    {
-      selectors = selectors.filter(function(s)
-      {
-        return !/\[style[\^\$]?=/.test(s);
-      });
-    }
-  }
+    selectors = ElemHide.getSelectorsForDomain(
+      extractHostFromFrame(sender.frame),
+      checkWhitelisted(sender.page, sender.frame,
+                       RegExpFilter.typeMap.GENERICHIDE)
+    );
+  else
+    selectors = [];
 
   return {selectors: selectors, trace: trace};
 });
 
 port.on("should-collapse", function(msg, sender)
 {
   if (checkWhitelisted(sender.page, sender.frame))
     return false;
 
   var typeMask = RegExpFilter.typeMap[msg.mediatype];
@@ skipping 49 matching lines @@
   }
 });
 
 // update icon when page changes location
 ext.pages.onLoading.addListener(function(page)
 {
   page.sendMessage({type: "composer.content.finished"});
   refreshIconAndContextMenu(page);
   showNextNotificationForUrl(page.url);
 });