Issue 3499 - Remove old messaging API

lib/main.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 20 matching lines @@
 require("sync");
 require("messageResponder");
 require("ui");
 require("objectTabs");
 
 function bootstrapChildProcesses()
 {
   let info = require("info");
 
   let processScript = info.addonRoot + "lib/child/bootstrap.js?" +
-      Math.random() + "#" + encodeURIComponent(JSON.stringify(info));

[Thomas Greiner, 2016/03/23 14:09:20]

That looks a bit hacky and not easily extensible.

Why not use URL query string parameters instead? `URLSearchParams` might help
making the parsing of those less tedious.

[Wladimir Palant, 2016/03/23 15:29:33]

I don't think extensibility matters here. I've implemented this as query
parameters now, not sure this really makes anything better.

+      Math.random() + "&info=" + encodeURIComponent(JSON.stringify(info));
   let messageManager = Cc["@mozilla.org/parentprocessmessagemanager;1"]
                          .getService(Ci.nsIProcessScriptLoader)
                          .QueryInterface(Ci.nsIMessageBroadcaster);
   messageManager.loadProcessScript(processScript, true);
 
   onShutdown.add(() => {
     messageManager.broadcastAsyncMessage("AdblockPlus:Shutdown", processScript);
     messageManager.removeDelayedProcessScript(processScript);
   });
 }
@@ skipping 21 matching lines @@
 
   let registrar = Components.manager.QueryInterface(Ci.nsIComponentRegistrar);
   registrar.registerFactory(classID, "Adblock Plus public API URL", contractID, factory);
 
   onShutdown.add(function()
   {
     registrar.unregisterFactory(classID, factory);
     Cu.unload(uri.spec);
   });
 }