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

Unified Diff: chrome/browser/ui/exclusive_access/fullscreen_controller_state_test.cc

Issue 789533002: Fullscreen: make fullscreen requests come from RenderFrame (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: make try happy Created 5 years, 11 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/exclusive_access/fullscreen_controller_state_test.cc
diff --git a/chrome/browser/ui/exclusive_access/fullscreen_controller_state_test.cc b/chrome/browser/ui/exclusive_access/fullscreen_controller_state_test.cc
index b2c3fb15df215eb5d6446f4d30401b7e152bf439..189b6548fe8f421be82d0ebd0ce91b0247177286 100644
--- a/chrome/browser/ui/exclusive_access/fullscreen_controller_state_test.cc
+++ b/chrome/browser/ui/exclusive_access/fullscreen_controller_state_test.cc
@@ -384,8 +384,13 @@ bool FullscreenControllerStateTest::InvokeEvent(Event event) {
case TAB_FULLSCREEN_FALSE: {
content::WebContents* const active_tab =
GetBrowser()->tab_strip_model()->GetActiveWebContents();
- GetFullscreenController()->
- ToggleFullscreenModeForTab(active_tab, event == TAB_FULLSCREEN_TRUE);
+ if (event == TAB_FULLSCREEN_TRUE) {
+ GetFullscreenController()->EnterFullscreenModeForTab(active_tab,
+ GURL());
+ } else {
+ GetFullscreenController()->ExitFullscreenModeForTab(active_tab);
+ }
+
// Activating/Deactivating tab fullscreen on a captured tab should not
// evoke a state change in the browser window.
if (active_tab->GetCapturerCount() > 0)

Powered by Google App Engine
This is Rietveld 408576698