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

Unified Diff: chrome/browser/ui/browser_browsertest.cc

Issue 7461059: Fullscreen JS API implementation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove commented out code Created 9 years, 4 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/browser/ui/browser_browsertest.cc
diff --git a/chrome/browser/ui/browser_browsertest.cc b/chrome/browser/ui/browser_browsertest.cc
index ff69a6624b5fd60fad8843e3b7a80d97341b58e1..859d686d9944a8031848f56fbe90381b0dfa705f 100644
--- a/chrome/browser/ui/browser_browsertest.cc
+++ b/chrome/browser/ui/browser_browsertest.cc
@@ -635,6 +635,23 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, MAYBE_PageLanguageDetection) {
EXPECT_EQ("fr", helper->language_state().original_language());
}
+IN_PROC_BROWSER_TEST_F(BrowserTest, TestToggleFullscreenModeForTab) {
+ ASSERT_TRUE(test_server()->Start());
+
+ AddTabAtIndex(0, GURL("about:blank"), PageTransition::TYPED);
+
+ TabContents* fullscreen_tab = browser()->GetSelectedTabContents();
+ TabContentsWrapper* tab_wrapper = browser()->GetSelectedTabContentsWrapper();
+
+ browser()->ToggleFullscreenModeForTab(fullscreen_tab, true);
Ben Goodger (Google) 2011/08/24 18:21:49 Can you add a test for exiting fullscreen mode usi
koz (OOO until 15th September) 2011/08/24 21:39:33 Done.
+ ui_test_utils::WaitForNotification(chrome::NOTIFICATION_FULLSCREEN_CHANGED);
+
+ ASSERT_TRUE(browser()->window()->IsFullscreen());
+ AddTabAtIndex(1, GURL("about:blank"), PageTransition::TYPED);
+ ui_test_utils::WaitForNotification(chrome::NOTIFICATION_FULLSCREEN_CHANGED);
+ ASSERT_FALSE(browser()->window()->IsFullscreen());
+}
+
// Chromeos defaults to restoring the last session, so this test isn't
// applicable.
#if !defined(OS_CHROMEOS)

Powered by Google App Engine
This is Rietveld 408576698