Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: chrome/background.js

Issue 5203617959116800: Fixed regression: Context menu clickhide broken (Closed)
Patch Set: Created Nov. 28, 2013, 3:08 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « background.js ('k') | safari/background.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/background.js
===================================================================
--- a/chrome/background.js
+++ b/chrome/background.js
@@ -350,4 +350,20 @@
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
Thomas Greiner 2013/12/09 16:33:48 Done.
+ }
+ });
+ },
+ removeAll: function(callback) {
+ chrome.contextMenus.removeAll(callback);
+ }
+ };
})();
« no previous file with comments | « background.js ('k') | safari/background.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld