More robust dynamic injection.

davireis · Sep 15, 2016 · 450731e · venkatkarun · Sep 15, 2016 · 450731e
1 parent cc4f38b
commit 450731e
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/jquery.dfp.js b/jquery.dfp.js
@@ -544,8 +544,15 @@
             };
 
             gads.src = 'https://www.googletagservices.com/tag/js/gpt.js';
+            // See https://www.stevesouders.com/blog/2010/05/11/appendchild-vs-insertbefore/
             var node = document.getElementsByTagName('script')[0];
-            node.parentNode.insertBefore(gads, node);
+            if (node) {
+                node.parentNode.insertBefore(gads, node);
+            } else {
+               // Sometimes there is no script tag due to dynamic injection,
+               // and then we fallback to a simpler method.
+               document.documentElement.firstChild.appendChild(gads);
+            }
 
             // Adblock plus seems to hide blocked scripts... so we check for that
             if (gads.style.display === 'none') {