Issue 6441 - Avoid unnecessary shadow root

include.preload.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present 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 346 matching lines @@
 
   createShadowTree()
   {
     // 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.

[Sebastian Noack, 2018/03/08 23:22:59]

This is irrelevant. We have feature detection above, and with the change to
<content>, it should be compatible with Firefox as well. The important point,
however, is that we want to avoid creating an unnecessary shadow root, if we can
use user style sheets.

[Manish Jethani, 2018/03/08 23:57:51]

You're right, I've updated the comment.

-    let {application, applicationVersion} = require("info");

[Sebastian Noack, 2018/03/08 23:22:59]

Do we use the "info" module in any other content script? If it's just for the
check here, I'm not sure if it justifies having it added to the bundle.

[Manish Jethani, 2018/03/08 23:57:51]

We don't. OK, I'm checking the navigator.userAgent property directly now.

-    if (application != "chrome" || parseInt(applicationVersion) > 65)

[Sebastian Noack, 2018/03/08 23:22:59]

We don't care about the application, but about the platform.

[Manish Jethani, 2018/03/08 23:57:51]

Acknowledged.

+    // 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.
@@ skipping 166 matching lines @@
     let element = event.target;
     if (/^i?frame$/.test(element.localName))
       checkCollapse(element);
   }, true);
 }
 
 window.checkCollapse = checkCollapse;
 window.elemhide = elemhide;
 window.typeMap = typeMap;
 window.getURLsFromElement = getURLsFromElement;