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

Unified Diff: chrome/test/data/extensions/api_test/browser_action/open_popup/open_popup_fails.js

Issue 2017113002: [Extensions] DCHECK that ExtensionFunctions respond (and only once) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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/browser_action/open_popup/open_popup_fails.js
diff --git a/chrome/test/data/extensions/api_test/browser_action/open_popup/open_popup_fails.js b/chrome/test/data/extensions/api_test/browser_action/open_popup/open_popup_fails.js
index 68116a2f0a14e5a94047cf8d84ad802505e33ec9..5a53ad78d7b56e5ba06edfd09ba534157cb7c6c5 100644
--- a/chrome/test/data/extensions/api_test/browser_action/open_popup/open_popup_fails.js
+++ b/chrome/test/data/extensions/api_test/browser_action/open_popup/open_popup_fails.js
@@ -7,7 +7,9 @@
// extension hides any open popups. After a popup is opened, we can assert that
// a popup fails to open with this API.
chrome.test.notifyPass();
-chrome.test.sendMessage('ready', function() {
+chrome.test.sendMessage('ready', function(reply) {
+ if (reply !== 'show another')
+ return;
chrome.browserAction.openPopup(function(popupWindow2) {
// This popup should fail to open.
chrome.test.assertTrue(!popupWindow2);

Powered by Google App Engine
This is Rietveld 408576698