Fixed regression: Context menu clickhide broken

The context menu initiated clickhide was broken due to the introduction of ext.*

This review introduces ext.contextMenus and the Chrome implementation of it to fix the regression. There will be a separate review for the Safari implementation due to issues that need to be addressed first.

[Sebastian Noack, 2013-11-28 23:17:40]

http://codereview.adblockplus.org/5203617959116800/diff/5629499534213120/chro...
File chrome/background.js (right):

http://codereview.adblockplus.org/5203617959116800/diff/5629499534213120/chro...
chrome/background.js:361: onclick(info, new Tab(tab));
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.

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

http://codereview.adblockplus.org/5203617959116800/diff/5629499534213120/chro...
File chrome/background.js (right):

http://codereview.adblockplus.org/5203617959116800/diff/5629499534213120/chro...
chrome/background.js:355: create: function(title, contexts, onclick) {
{ should go on a new line, likewise below.

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

http://codereview.adblockplus.org/5203617959116800/diff/5629499534213120/chro...
File chrome/background.js (right):

http://codereview.adblockplus.org/5203617959116800/diff/5629499534213120/chro...
chrome/background.js:355: create: function(title, contexts, onclick) {
On 2013/12/04 15:24:02, Felix H. Dahlke wrote:
> { should go on a new line, likewise below.

Done.

http://codereview.adblockplus.org/5203617959116800/diff/5629499534213120/chro...
chrome/background.js:361: onclick(info, new Tab(tab));
On 2013/11/28 23:17:41, sebastian wrote:
> 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.

Done.

[Wladimir Palant, 2013-12-16 10:38:23]

LGTM

[Sebastian Noack, 2013-12-16 13:32:36]

LGTM

[Felix Dahlke, 2013-12-17 15:39:51]

LGTM