Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(220)

Unified Diff: chrome/test/data/extensions/api_test/context_menus/event_page/test.js

Issue 10454106: Dispatch a new event chrome.contextMenus.onClicked. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: names Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/extensions/api_test/context_menus/event_page/test.js
diff --git a/chrome/test/data/extensions/api_test/context_menus/event_page_item_ids/test.js b/chrome/test/data/extensions/api_test/context_menus/event_page/test.js
similarity index 67%
rename from chrome/test/data/extensions/api_test/context_menus/event_page_item_ids/test.js
rename to chrome/test/data/extensions/api_test/context_menus/event_page/test.js
index 119fd0c406aabd577cb859f7e60aa2baf884c933..930b5e4d91f94f6c47c604819bfc642b51ebd1bf 100644
--- a/chrome/test/data/extensions/api_test/context_menus/event_page_item_ids/test.js
+++ b/chrome/test/data/extensions/api_test/context_menus/event_page/test.js
@@ -17,5 +17,14 @@ chrome.test.runTests([
{"title": "title2"},
chrome.test.callbackFail("Extensions using event pages must pass an " +
"id parameter to chrome.contextMenus.create"));
+ },
+
+ function noOnClick() {
+ chrome.contextMenus.create(
+ {"id": "id3", "title": "title3", "onclick": function() {}},
+ chrome.test.callbackFail(
+ "Extensions using event pages cannot pass an onclick parameter " +
+ "to chrome.contextMenus.create. Instead, use the " +
+ "chrome.contextMenus.onClicked event."));
}
]);

Powered by Google App Engine
This is Rietveld 408576698