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: Use Shadow DOM v0 only on Chrome 65 and lower Created March 8, 2018, 9:50 p.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,16 +359,24 @@
{
// 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;
+ // Use Shadow DOM only on Chrome version 65 and lower. Firefox's Shadow
+ // DOM v0 implementation is slightly different (e.g. no <shadow> element),
+ // while Chrome is deprecating Shadow DOM v0 in favor of Shadow DOM v1 and
+ // user style sheets.
+ let {application, applicationVersion} = require("info");
+ if (application != "chrome" || parseInt(applicationVersion) > 65)
+ 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.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld