Issue 2879 - Move element selection into the content process

lib/windowWrapper.js

 /*
  * This Source Code is subject to the terms of the Mozilla Public License
  * version 2.0 (the "License"). You can obtain a copy of the License at
  * http://mozilla.org/MPL/2.0/.
  */
 
 let {Aardvark} = require("aardvark");
 let {Prefs} = require("prefs");
 let {KeySelector} = require("keySelector");
 
@@ skipping 62 matching lines @@
   },
 
   popupShowingHandler: function(event)
   {
     let popup = event.originalTarget;
     if (!/^(abp-(?:toolbar|status|menuitem)-)popup$/.test(popup.id))
       return;
 
     this.popupHiddenHandler(event);
 
-    let enabled = Aardvark.canSelect(this.browser);
-    let running = (enabled && this.browser == Aardvark.browser);
+    let running = this.browser == Aardvark.browser;
 
     let [labelStart, labelStop] = getMenuItem();
     let item = popup.ownerDocument.createElement("menuitem");
     item.setAttribute("label", running ? labelStop : labelStart);
     item.setAttribute("class", "elemhidehelper-item");
-    if (!enabled)
-      item.setAttribute("disabled", "true");

[Wladimir Palant, 2016/11/17 13:54:48]

The menu item used to indicate if Element Hiding Helper wouldn't run on some
page. Now that canSelect moved to the child process this got somewhat more
complicated and removed that indicator for now. I think that I will need to
bring it back - show the menu as enabled initially, and set "disabled" attribute
after receiving a response from child.

[saroyanm, 2016/11/23 17:44:39]

Acknowledged.

 
     if (typeof key == "undefined")
       this.configureKey(event.currentTarget);
     item.setAttribute("acceltext", KeySelector.getTextForKey(key));
 
     item.addEventListener("command", this.toggleSelection, false);
 
     let insertBefore = null;
     for (let child = popup.firstChild; child; child = child.nextSibling)
       if (/-options$/.test(child.id))
@@ skipping 33 matching lines @@
   {
     if (Aardvark.window == this.window)
       Aardvark.hideTooltips();
   },
 
   toggleSelection: function()
   {
     if ("@adblockplus.org/abp/public;1" in Cc && this.browser != Aardvark.browser)
       Aardvark.start(this);
     else
-      Aardvark.quit();
+      Aardvark.doCommand("quit", null);
   }
 };