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

Unified Diff: chrome/test/data/extensions/api_test/toolstrip/test.html

Issue 195093: handle mole/toolstrip URLs properly (Closed)
Patch Set: Created 11 years, 3 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/toolstrip/test.html
diff --git a/chrome/test/data/extensions/api_test/toolstrip/test.html b/chrome/test/data/extensions/api_test/toolstrip/test.html
index 237612dfba7103f9dfe5d5e11cc203742cce5e3d..985e751c78ff5d595b58dbcc5c660499decd2323 100755
--- a/chrome/test/data/extensions/api_test/toolstrip/test.html
+++ b/chrome/test/data/extensions/api_test/toolstrip/test.html
@@ -9,6 +9,34 @@ chrome.test.runTests([
function collapse() {
chrome.test.listenOnce(chrome.toolstrip.onCollapsed, function(){});
chrome.toolstrip.collapse({}, chrome.test.callbackPass(function(){}));
- }
+ },
+
+ /* TODO(mpcomplete): this is failing in part due to bug 21905
+ function expandOther() {
+ var orig = window.location;
+ var relative = "folder/relative.html";
+
+ // Get the other toolstrip.
+ var toolstrips = chrome.extension.getToolstrips();
+ var other = (toolstrips[0].location == location) ?
+ toolstrips[1] : toolstrips[0];
+
+ other.chrome.toolstrip.expand({height:200, url:relative},
+ chrome.test.callbackPass(function() {
+ // TODO(mpcomplete) this never gets called
+ chrome.test.log(other.location.toString());
+ chrome.test.listenOnce(other.onLoad, function() {
+ chrome.test.log(other.location.toString());
+ chrome.test.assertEq(other.location.toString(),
+ chrome.extension.getURL(relative));
+ other.chrome.toolstrip.collapse({url:"../test2.html"},
+ chrome.test.callbackPass(function() {
+ chrome.test.assertEq(other.location.toString(), orig);
+ }));
+ });
+ }));
+ },
+ */
]);
</script>
+test

Powered by Google App Engine
This is Rietveld 408576698