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

Unified Diff: chrome/test/data/extensions/api_test/window_update/show_state/test.js

Issue 9328037: Add fullscreen state support for chrome.windows (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Small fix Created 8 years, 10 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/window_update/show_state/test.js
diff --git a/chrome/test/data/extensions/api_test/window_update/show_state/test.js b/chrome/test/data/extensions/api_test/window_update/show_state/test.js
index e0496df52c83d085315052dd3861bc1dea63a20b..ab01690279ecb6b77909be18b3e6d63ca2131868 100644
--- a/chrome/test/data/extensions/api_test/window_update/show_state/test.js
+++ b/chrome/test/data/extensions/api_test/window_update/show_state/test.js
@@ -17,23 +17,37 @@ function checkRestoreWithBounds(theWindow) {
chrome.windows.remove(theWindow.id, pass());
}
+function checkFullscreen(theWindow) {
+ chrome.test.assertEq('fullscreen', theWindow.state);
+ chrome.test.assertTrue(width < theWindow.width || height < theWindow.height);
+
+ width += deltaWidth;
+ height += deltaHeight;
+ chrome.windows.update(theWindow.id,
+ {'state': 'normal', 'width': width, 'height': height},
+ pass(checkRestoreWithBounds));
+}
+
function checkMaximized(theWindow) {
if (theWindow.type == 'panel') {
// Maximize is the same as restore for panels.
chrome.test.assertEq('normal', theWindow.state);
chrome.test.assertEq(width, theWindow.width);
chrome.test.assertEq(height, theWindow.height);
+
+ width += deltaWidth;
+ height += deltaHeight;
+ chrome.windows.update(theWindow.id,
+ {'state': 'normal', 'width': width, 'height': height},
+ pass(checkRestoreWithBounds));
jennb 2012/02/06 23:34:25 Instead of having this special test path for panel
hashimoto 2012/02/07 02:16:00 Done. I made no assertion for panels because Pane
} else {
chrome.test.assertEq('maximized', theWindow.state);
chrome.test.assertTrue(width < theWindow.width ||
height < theWindow.height);
- }
- width += deltaWidth;
- height += deltaHeight;
- chrome.windows.update(theWindow.id,
- {'state': 'normal', 'width': width, 'height': height},
- pass(checkRestoreWithBounds));
+ chrome.windows.update(theWindow.id, {'state': 'fullscreen'},
+ pass(checkFullscreen));
+ }
}
function checkRestored(theWindow) {

Powered by Google App Engine
This is Rietveld 408576698