Issue 4361 - Pass all parameters to initEvent

safari/ext/content.js

Index: safari/ext/content.js
diff --git a/safari/ext/content.js b/safari/ext/content.js
index 2c43b2dbef79aed4fb2a5458fcceea6fbb13a341..488abc7083ad92d39c4b04e2042a71aa4a2e16af 100644
--- a/safari/ext/content.js
+++ b/safari/ext/content.js
@@ -26,7 +26,7 @@
   /* Intialization */
 
   var beforeLoadEvent = document.createEvent("Event");
-  beforeLoadEvent.initEvent("beforeload");
+  beforeLoadEvent.initEvent("beforeload", false, true);
 
   // Decide if we should use the new content blocker API or not. (Note when the
   // API is used Safari breaks the canLoad function, making it either throw an
@@ -153,7 +153,7 @@
         setTimeout(function()
         {
           var evt = document.createEvent("Event");
-          evt.initEvent(eventName);
+          evt.initEvent(eventName, false, true);

[Sebastian Noack, 2016/09/16 12:42:52]

"load" and "error" event don't bubble. I think "beforeload" (above) doesn't
buuble either,but feel free to double check.

[Sebastian Noack, 2016/09/16 12:48:52]

Sorry, got it the wrong way around, the second argument (bubbles) is actually
correctly set to false. However, "load" and "error" events aren't cancelable, so
the third argument needs to be set to false too, here.

[kzar, 2016/09/16 13:00:43]

Done.

           event.target.dispatchEvent(evt);
         });
       }