Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: include.preload.js

Issue 29714555: Issue 6441 - Avoid unnecessary shadow root (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: Improve user agent check Created March 9, 2018, 12:32 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include.preload.js
===================================================================
--- a/include.preload.js
+++ b/include.preload.js
@@ -359,27 +359,33 @@
{
// Use Shadow DOM if available as to not mess with with web pages that
// rely on the order of their own <style> tags (#309). However, creating
// a shadow root breaks running CSS transitions. So we have to create
// the shadow root before transistions might start (#452).
if (!("createShadowRoot" in document.documentElement))
return null;
+ // Both Firefox and Chrome 66+ support user style sheets, so we can avoid
+ // creating an unnecessary shadow root on these platforms.
+ let match = /\bChrome\/(\d+)/.exec(navigator.userAgent);
+ if (!match || match[1] >= 66)
+ return null;
+
// Using shadow DOM causes issues on some Google websites,
// including Google Docs, Gmail and Blogger (#1770, #2602, #2687).
if (/\.(?:google|blogger)\.com$/.test(document.domain))
return null;
// Finally since some users have both AdBlock and Adblock Plus installed we
// have to consider how the two extensions interact. For example we want to
// avoid creating the shadowRoot twice.
let shadow = document.documentElement.shadowRoot ||
document.documentElement.createShadowRoot();
- shadow.appendChild(document.createElement("shadow"));
+ shadow.appendChild(document.createElement("content"));
return shadow;
},
addSelectorsInline(selectors, groupName)
{
let style = this.styles.get(groupName);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld