Issue 1703 - Work around Chrome bug 442107, hiding placeholders in frames with javascript: URL

include.preload.js

Index: include.preload.js
===================================================================
--- a/include.preload.js
+++ b/include.preload.js
@@ -157,12 +157,6 @@
 
 function init(document)
 {
-  // prior to Chrome 37, content scripts don't run on about:blank
-  // and about:srcdoc. So we have to apply element hiding and collapsing
-  // from the parent frame, when inline frames are loaded.
-  var match = navigator.userAgent.match(/\bChrome\/(\d+)/);
-  var fixInlineFrames = match && parseInt(match[1], 10) < 37;
-
   // use Shadow DOM if available to don't mess with web pages that
   // rely on the order of their own <style> tags (#309). However we
   // must not create the shadow root in the response callback passed
@@ -233,7 +227,11 @@
     if (/^i?frame$/.test(element.localName))
       checkCollapse(element);
 
-    if (fixInlineFrames && isInlineFrame(element))
+    // prior to Chrome 37, content scripts cannot run on about:blank,
+    // about:srcdoc and javascript: URLs. Moreover, as of Chrome 40
+    // "load" and "error" events aren't dispatched there. So we have
+    // to apply element hiding and collapsing from the parent frame.
+    if (/\bChrome\//.test(navigator.userAgent) && isInlineFrame(element))
     {
       init(element.contentDocument);