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

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

Issue 11896104: Refactor FullscreenController removing TogglePresentationMode & adding ToggleFullscreenWithChrome. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge TOT Created 7 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/browser_command_controller.cc
diff --git a/chrome/browser/ui/browser_command_controller.cc b/chrome/browser/ui/browser_command_controller.cc
index 72ce592df0f30d8f04bedba3729052a0a7ff884d..892aef40dcb6523acbdebb9bb36c51473b2a62e8 100644
--- a/chrome/browser/ui/browser_command_controller.cc
+++ b/chrome/browser/ui/browser_command_controller.cc
@@ -421,7 +421,11 @@ void BrowserCommandController::ExecuteCommandWithDisposition(
ConvertPopupToTabbedBrowser(browser_);
break;
case IDC_FULLSCREEN:
+#if defined(OS_MACOSX)
+ chrome::ToggleFullscreenWithChrome(browser_);
+#else
chrome::ToggleFullscreenMode(browser_);
+#endif
break;
#if defined(USE_ASH)
@@ -450,7 +454,7 @@ void BrowserCommandController::ExecuteCommandWithDisposition(
#if defined(OS_MACOSX)
case IDC_PRESENTATION_MODE:
- browser_->TogglePresentationMode();
+ chrome::ToggleFullscreenMode(browser_);
break;
#endif
case IDC_EXIT:

Powered by Google App Engine
This is Rietveld 408576698