Issue 3200 - Delay initialization for prerendered documents

safari/ext/background.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 165 matching lines @@
   /* Pages */
 
   var pages = Object.create(null);
   var pageCounter = 0;
 
   var Page = function(id, tab, url)
   {
     this.id = id;
     this._tab = tab;
     this._frames = [{url: new URL(url), parent: null}];
+    this._messageProxy = null;
 
     this.browserAction = new BrowserAction(this);
     this.contextMenus = new ContextMenus(this);

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

I guess we should assign this._messageProxy to undefined (or null?) here
explicitly, since the modified logic in _getMessageProxy will sometimes return
that?

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

Yeah, it'll always return undefined or a non-null object. I think I'll just
assign it to null just to make it clear.

   };
   Page.prototype = {
     get url()
     {
       return this._frames[0].url;
     },
     sendMessage: function(message, responseCallback)
     {
       var documentIds = [];
       for (var documentId in this._tab._documentLookup)
@@ skipping 410 matching lines @@
   /* Windows */
   ext.windows = {
     // Safari doesn't provide as rich a windows API as Chrome does, so instead
     // of chrome.windows.create we have to fall back to just opening a new tab.
     create: function(createData, callback)
     {
       ext.pages.open(createData.url, callback);
     }
   };
 })();