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 334 matching lines @@
     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.

+    create: function(title, contexts, onclick)
+    {
       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.

+          onclick(info.srcUrl, new Tab(tab));
         }
       });
     },
-    removeAll: function(callback) {
+    removeAll: function(callback)
+    {
       chrome.contextMenus.removeAll(callback);
     }
   };
 })();