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

Unified Diff: chrome/test/data/extensions/api_test/popup_api/toolband.html

Issue 385061: experimental.popup support for tab-content-viewed extensions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 1 month 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/popup_api/toolband.html
===================================================================
--- chrome/test/data/extensions/api_test/popup_api/toolband.html (revision 31907)
+++ chrome/test/data/extensions/api_test/popup_api/toolband.html (working copy)
@@ -1,5 +1,6 @@
+<html xmlns="http://www.w3.org/1999/xhtml">
+<head>
<script>
-
var globalValue = "I am not 42.";
window.onload = function() {
@@ -42,16 +43,20 @@
function closePopup() {
chrome.test.listenOnce(chrome.experimental.popup.onClosed, function(){
- chrome.test.assertTrue(
- chrome.experimental.extension.getPopupView() == undefined);
+ // TODO(twiz): getPopupView() should return undefined, but, due to
+ // shut-down races, it is sometimes still defined. See BUG
+ //chrome.test.assertTrue(
+ // chrome.experimental.extension.getPopupView() == undefined);
});
chrome.experimental.extension.getPopupView().close();
}
]);
}
</script>
+</head>
<body>
<div>
<span id="anchorHere">TEST</span>
</div>
</body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698