Issue 3200 - Delay initialization for prerendered documents

composer.postload.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
- * Copyright (C) 2006-2017 eyeo GmbH
+ * 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
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
@@ skipping 562 matching lines @@
               type: "composer.content.finished"
             }
           });
         }
         break;
     }
   });
 
   if (window == window.top)
   {
-    if (document.visibilityState == "prerender")
-    {
-      document.addEventListener("visibilitychange", function onVisibilitychange()

[kzar, 2017/05/19 12:33:05]

Nit: Long line.

Also it's kind of unusual how you've put the named function inside the call to
addEventListener and then referenced that name from inside the callback. Mind
putting it above instead? (Same in include.preload.js.)

[Manish Jethani, 2017/05/19 23:58:48]

Sure, I wasn't sure what the preferred style was.

-      {
-        document.removeEventListener("visibilitychange", onVisibilitychange);
-
-        ext.backgroundPage.sendMessage({type: "composer.ready"});
-      });
-    }
-    else
+    whenDocumentVisible(() =>
     {
       ext.backgroundPage.sendMessage({type: "composer.ready"});
-    }
-  }
-}
+    });
+  }
+}