Issue 375 - Make context menus per page

safari/ext/background.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2014 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 23 matching lines @@
     else
       // while the new tab page is shown on Safari 7, the 'page' property
       // of the tab is undefined, and we can't send messages to that page
       this._messageProxy = {
         handleRequest: function() {},
         handleResponse: function() {},
         sendMessage: function() {}
       };
 
     this.browserAction = new BrowserAction(this);
+    this.contextMenus = new ContextMenus(this);
   };
   Page.prototype = {
     get url()
     {
       return this._frames[0].url;
     },
     activate: function()
     {
       this._tab.activate();
     },
@@ skipping 175 matching lines @@
       {
         activePage = page;
         break;
       }
     }
 
     updateToolbarItemForPage(activePage, event.target.browserWindow);
   }, true);
 
 
-  /* Web requests */
-
-  ext.webRequest = {
-    onBeforeRequest: new ext._EventTarget(true),
-    handlerBehaviorChanged: function() {}
-  };
-
-
   /* Context menus */
 
-  var contextMenuItems = [];
-  var isContextMenuHidden = true;
+  var contextMenuItems = new ext.PageMap();
 
-  ext.contextMenus = {
-    addMenuItem: function(title, contexts, onclick)
+  var ContextMenus = function(page)
+  {
+    this._page = page;
+  };
+  ContextMenus.prototype = {
+    create: function(item)
     {
-      contextMenuItems.push({
-        id: String(contextMenuItems.length),
-        title: title,
-        item: null,
-        contexts: contexts,
-        onclick: onclick
-      });
-      this.showMenuItems();
+      var items = contextMenuItems.get(this._page);
+      if (!items)
+        contextMenuItems.set(this._page, items = []);
+
+      items.push(item);
     },
-    removeMenuItems: function()
+    removeAll: function()
     {
-      contextMenuItems = [];
-      this.hideMenuItems();
-    },
-    showMenuItems: function()
-    {
-      isContextMenuHidden = false;
-    },
-    hideMenuItems: function()
-    {
-      isContextMenuHidden = true;
+      contextMenuItems.delete(this._page);
     }
   };
 
   safari.application.addEventListener("contextmenu", function(event)
   {
-    if (isContextMenuHidden)
+    var pageId = event.userInfo.pageId;
+    if (!pageId)
+      return;
+
+    var page = pages[event.userInfo.pageId];
+    var items = contextMenuItems.get(page);
+    if (!items)
       return;
 
     var context = event.userInfo.tagName;
     if (context == "img")
       context = "image";
     if (!event.userInfo.srcUrl)
       context = null;
 
-    for (var i = 0; i < contextMenuItems.length; i++)
+    for (var i = 0; i < items.length; i++)
     {
       // Supported contexts are: all, audio, image, video
-      var menuItem = contextMenuItems[i];
+      var menuItem = items[i];
       if (menuItem.contexts.indexOf("all") == -1 && menuItem.contexts.indexOf(context) == -1)
         continue;
 
-      event.contextMenu.appendContextMenuItem(menuItem.id, menuItem.title);
+      event.contextMenu.appendContextMenuItem(i, menuItem.title);
     }
   });
 
   safari.application.addEventListener("command", function(event)
   {
-    for (var i = 0; i < contextMenuItems.length; i++)
-    {
-      if (contextMenuItems[i].id == event.command)
-      {
-        contextMenuItems[i].onclick(event.userInfo.srcUrl, pages[event.userInfo.pageId]);
-        break;
-      }
-    }
+    var page = pages[event.userInfo.pageId];
+    var items = contextMenuItems.get(page);
+
+    items[event.command].onclick(event.userInfo.srcUrl, page);

[Wladimir Palant, 2014/04/25 11:35:50]

Can there be unknown commands, e.g. from the browser itself?

[Sebastian Noack, 2014/04/25 11:48:03]

As far as I know the "command" event is only dispatched when a custom context
menu entry is clicked. I couldn't find any other case in the documentation and
during my tests.

   });
 
 
+  /* Web requests */
+
+  ext.webRequest = {
+    onBeforeRequest: new ext._EventTarget(true),
+    handlerBehaviorChanged: function() {}
+  };
+
+
   /* Background page */
 
   ext.backgroundPage = {
     getWindow: function()
     {
       return window;
     }
   };
 
 
@@ skipping 333 matching lines @@
         replacePage(page);
         break;
     }
   });
 
 
   /* Storage */
 
   ext.storage = safari.extension.settings;
 })();