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
  *
  * 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 564 matching lines @@
             {
               type: "composer.content.finished"
             }
           });
         }
         break;
     }
   });
 
   if (window == window.top)
-    ext.backgroundPage.sendMessage({type: "composer.ready"});
+  {
+    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
+    {
+      ext.backgroundPage.sendMessage({type: "composer.ready"});
+    }
+  }
 }