Unified Diff: include.preload.js
Issue 5838948538515456:
Issue 370 - Make "Block element" hide elements for added filters (Closed)
Patch Set: Added comments
Created March 4, 2015, 2 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« include.postload.js ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: include.preload.js |
| =================================================================== |
| --- a/include.preload.js |
| +++ b/include.preload.js |
| @@ -180,6 +180,9 @@ |
| function init(document) |
| { |
| + var shadow = null; |
| + var style = null; |
| + |
| // Use Shadow DOM if available to don't mess with web pages that rely on |
| // the order of their own <style> tags (#309). |
| // |
| @@ -188,57 +191,51 @@ |
| // |
| // Also, we can't use shadow DOM on Google Docs, since it breaks printing |
| // there (#1770). |
| - var shadow = null; |
| if ("createShadowRoot" in document.documentElement && document.domain != "docs.google.com") |
| { |
| shadow = document.documentElement.createShadowRoot(); |
| shadow.appendChild(document.createElement("shadow")); |
| } |
| - // Sets the currently used CSS rules for elemhide filters |
| - var setElemhideCSSRules = function(selectors) |
| + var hideElements = function(selectors) |
| { |
| - if (selectors.length == 0) |
| - return; |
| - |
| - var style = document.createElement("style"); |
| - style.setAttribute("type", "text/css"); |
| - |
| - if (shadow) |
| + // Create <style> element lazily, only if we add styles. Add it to |
| + // the shadow DOM if possible. Otherwise fallback to the <head> or |
| + // <html> element. If we have injected a style element before that |
| + // has been removed (the sheet property is null), create a new one. |
| + if (!style || !style.sheet) |
| { |
| - shadow.appendChild(style); |
| - selectors = convertSelectorsForShadowDOM(selectors); |
| - } |
| - else |
| - { |
| - // Try to insert the style into the <head> tag, inserting directly under the |
| - // document root breaks dev tools functionality: |
| - // http://code.google.com/p/chromium/issues/detail?id=178109 |
| - (document.head || document.documentElement).appendChild(style); |
| + style = document.createElement("style"); |
| + (shadow || document.head || document.documentElement).appendChild(style); |
| } |
| - var setRules = function() |
| + // If using shadow DOM, we have to add the ::content pseudo-element |
| + // before each selector, in order to match elements within the |
| + // insertion point. |
| + if (shadow) |
| + selectors = convertSelectorsForShadowDOM(selectors); |
| + |
| + // WebKit (and Blink?) apparently chokes when the selector list in a |
| + // CSS rule is huge. So we split the elemhide selectors into groups. |
| + while (selectors.length > 0) |
| { |
| - // The sheet property might not exist yet if the |
| - // <style> element was created for a sub frame |
| - if (!style.sheet) |
| - { |
| - setTimeout(setRules, 0); |
|
Wladimir Palant
2015/03/04 14:45:54
Why did you remove this failsafe? It was introduce
Sebastian Noack
2015/03/04 15:10:22
First I didn't remember why I added that code, and
Sebastian Noack
2015/03/04 18:55:10
I figured out the issue here. If adding a <style>
|
| - return; |
| - } |
| + var selector = selectors.splice(0, SELECTOR_GROUP_SIZE).join(", "); |
| - // WebKit apparently chokes when the selector list in a CSS rule is huge. |
| - // So we split the elemhide selectors into groups. |
| - for (var i = 0; selectors.length > 0; i++) |
| - { |
| - var selector = selectors.splice(0, SELECTOR_GROUP_SIZE).join(", "); |
| - style.sheet.insertRule(selector + " { display: none !important; }", i); |
| - } |
| - }; |
| + style.sheet.insertRule( |
| + selector + " { display: none !important; }", |
| + style.sheet.cssRules.length |
| + ); |
| + } |
| + }; |
| - setRules(); |
| - reinjectRulesWhenRemoved(document, style); |
| - }; |
| + ext.backgroundPage.sendMessage({type: "get-selectors"}, function(selectors) |
| + { |
| + if (selectors.length > 0) |
| + { |
| + hideElements(selectors); |
| + reinjectRulesWhenRemoved(document, style); |
| + } |
| + }); |
| document.addEventListener("error", function(event) |
| { |
| @@ -265,11 +262,11 @@ |
| } |
| }, true); |
| - ext.backgroundPage.sendMessage({type: "get-selectors"}, setElemhideCSSRules); |
| + return hideElements; |
| } |
| if (document instanceof HTMLDocument) |
| { |
| checkSitekey(); |
| - init(document); |
| + window.hideElements = init(document); |
| } |
« no previous file with comments
|
« include.postload.js ('k')
|
no next file »
|
no next file with comments »
