Issue 3964 - Fix race condition, CSS property filters applying too early

lib/child/cssProperties.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 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 44 matching lines @@
     Services.scriptloader.loadSubScript(
         "chrome://adblockplus/content/cssProperties.js", scope);
 
     let onContentWindow = (subject, topic, data) =>
     {
       if (!(subject instanceof Ci.nsIDOMWindow))
         return;
 
       let onReady = event =>
       {
-        subject.removeEventListener("DOMContentLoaded", onReady);
+        subject.removeEventListener("load", onReady);
         let handler = new scope.CSSPropertyFilters(subject, selectors =>
         {
           if (selectors.length == 0)
             return;
 
           addUserCSS(subject, selectors.map(
             selector => selector + "{display: none !important;}"
           ).join("\n"));
         });
 
         // HACK: The content script just calls ext.backgroundPage.sendMessage
         // without indicating which window this is about. We'll store the window
         // here because we know that messaging happens synchronously.
         senderWindow = subject;
         handler.load(() => handler.apply());
         senderWindow = null;
       };
 
-      subject.addEventListener("DOMContentLoaded", onReady);
+      subject.addEventListener("load", onReady);
     };
 
     Services.obs.addObserver(onContentWindow, "content-document-global-created",
         false);
     onShutdown.add(() =>
     {
       Services.obs.removeObserver(onContentWindow,
           "content-document-global-created");
     });
   }
 
   initCSSPropertyFilters();
 })();