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

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: Separate FullscreenController related stuff to another patch set 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
« no previous file with comments | « chrome/common/extensions/docs/windows.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..0beb5f47af0366013deed88a14416775c12dac25 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
@@ -10,13 +10,29 @@ var height = 0;
var deltaWidth = 20;
var deltaHeight = 30;
-function checkRestoreWithBounds(theWindow) {
+function checkRestoreAfterFullscreen(theWindow) {
chrome.test.assertEq('normal', theWindow.state);
chrome.test.assertEq(width, theWindow.width);
chrome.test.assertEq(height, theWindow.height);
chrome.windows.remove(theWindow.id, pass());
}
+function checkFullscreen(theWindow) {
+ if (theWindow.type != 'panel')
jennb 2012/02/07 18:41:38 Can you set the panel case to check that the state
hashimoto 2012/02/08 04:22:35 Done.
+ chrome.test.assertEq('fullscreen', theWindow.state);
hashimoto 2012/02/07 11:53:17 Not checking size here because BrowserWindowCocoa
+ chrome.windows.update(theWindow.id, {'state': 'normal'},
+ pass(checkRestoreAfterFullscreen));
+}
+
+function checkRestoreWithBounds(theWindow) {
+ chrome.test.assertEq('normal', theWindow.state);
+ chrome.test.assertEq(width, theWindow.width);
+ chrome.test.assertEq(height, theWindow.height);
+
+ chrome.windows.update(theWindow.id, {'state': 'fullscreen'},
+ pass(checkFullscreen));
+}
+
function checkMaximized(theWindow) {
if (theWindow.type == 'panel') {
// Maximize is the same as restore for panels.
« no previous file with comments | « chrome/common/extensions/docs/windows.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698