Fixed regression: Context menu clickhide broken

chrome/background.js

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2013 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 332 matching lines @@
     onLoading: new LoadingTabEventTarget(),
     onCompleted: new CompletedTabEventTarget(),
     onActivated: new ActivatedTabEventTarget(),
     onRemoved: new RemovedTabEventTarget()
   };
 
   ext.webRequest = {
     onBeforeRequest: new BeforeRequestEventTarget(),
     handlerBehaviorChanged: chrome.webRequest.handlerBehaviorChanged
   };
+  
+  ext.contextMenus = {
+    create: function(title, contexts, onclick) {

[Felix Dahlke, 2013/12/04 15:24:02]

{ should go on a new line, likewise below.

[Thomas Greiner, 2013/12/09 16:33:48]

Done.

+      chrome.contextMenus.create({
+        title: title,
+        contexts: contexts,
+        onclick: function(info, tab)
+        {
+          onclick(info, new Tab(tab));

[Sebastian Noack, 2013/11/28 23:17:41]

The OnClickData is a rich data structue with some data that aren't available on
Safari or can only provided with some additional effort. So I suggest instead of
forwarding the OnClickData, to only pass the data that we actually need, which
seems to be only the srcUrl.

[Thomas Greiner, 2013/12/09 16:33:48]

Done.

+        }
+      });
+    },
+    removeAll: function(callback) {
+      chrome.contextMenus.removeAll(callback);
+    }
+  };
 })();