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

Side by Side Diff: chrome/browser/ui/browser_command_controller.cc

Issue 1414033009: Implement a Menu Item to Hide the Toolbar in Fullscreen (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase and merge code Created 5 years, 1 month 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 unified diff | Download patch
« no previous file with comments | « chrome/app/nibs/MainMenu.xib ('k') | chrome/browser/ui/browser_commands_mac.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/browser_command_controller.h" 5 #include "chrome/browser/ui/browser_command_controller.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/prefs/pref_service.h" 8 #include "base/prefs/pref_service.h"
9 #include "chrome/app/chrome_command_ids.h" 9 #include "chrome/app/chrome_command_ids.h"
10 #include "chrome/browser/browser_process.h" 10 #include "chrome/browser/browser_process.h"
(...skipping 508 matching lines...) Expand 10 before | Expand all | Expand 10 after
519 break; 519 break;
520 case IDC_PIN_TO_START_SCREEN: 520 case IDC_PIN_TO_START_SCREEN:
521 TogglePagePinnedToStartScreen(browser_); 521 TogglePagePinnedToStartScreen(browser_);
522 break; 522 break;
523 #endif 523 #endif
524 524
525 #if defined(OS_MACOSX) 525 #if defined(OS_MACOSX)
526 case IDC_PRESENTATION_MODE: 526 case IDC_PRESENTATION_MODE:
527 chrome::ToggleFullscreenMode(browser_); 527 chrome::ToggleFullscreenMode(browser_);
528 break; 528 break;
529 case IDC_TOGGLE_FULLSCREEN_TOOLBAR:
530 chrome::ToggleFullscreenToolbar(browser_);
531 break;
529 #endif 532 #endif
530 case IDC_EXIT: 533 case IDC_EXIT:
531 Exit(); 534 Exit();
532 break; 535 break;
533 536
534 // Page-related commands 537 // Page-related commands
535 case IDC_SAVE_PAGE: 538 case IDC_SAVE_PAGE:
536 SavePage(browser_); 539 SavePage(browser_);
537 break; 540 break;
538 case IDC_BOOKMARK_PAGE: 541 case IDC_BOOKMARK_PAGE:
(...skipping 746 matching lines...) Expand 10 before | Expand all | Expand 10 after
1285 if (window_state == WINDOW_STATE_NOT_FULLSCREEN && 1288 if (window_state == WINDOW_STATE_NOT_FULLSCREEN &&
1286 !profile()->GetPrefs()->GetBoolean(prefs::kFullscreenAllowed)) { 1289 !profile()->GetPrefs()->GetBoolean(prefs::kFullscreenAllowed)) {
1287 // Disable toggling into fullscreen mode if disallowed by pref. 1290 // Disable toggling into fullscreen mode if disallowed by pref.
1288 fullscreen_enabled = false; 1291 fullscreen_enabled = false;
1289 } 1292 }
1290 #endif 1293 #endif
1291 1294
1292 command_updater_.UpdateCommandEnabled(IDC_FULLSCREEN, fullscreen_enabled); 1295 command_updater_.UpdateCommandEnabled(IDC_FULLSCREEN, fullscreen_enabled);
1293 command_updater_.UpdateCommandEnabled(IDC_PRESENTATION_MODE, 1296 command_updater_.UpdateCommandEnabled(IDC_PRESENTATION_MODE,
1294 fullscreen_enabled); 1297 fullscreen_enabled);
1298 command_updater_.UpdateCommandEnabled(IDC_TOGGLE_FULLSCREEN_TOOLBAR,
1299 fullscreen_enabled);
1295 1300
1296 UpdateCommandsForBookmarkBar(); 1301 UpdateCommandsForBookmarkBar();
1297 } 1302 }
1298 1303
1299 void BrowserCommandController::UpdatePrintingState() { 1304 void BrowserCommandController::UpdatePrintingState() {
1300 bool print_enabled = CanPrint(browser_); 1305 bool print_enabled = CanPrint(browser_);
1301 command_updater_.UpdateCommandEnabled(IDC_PRINT, print_enabled); 1306 command_updater_.UpdateCommandEnabled(IDC_PRINT, print_enabled);
1302 #if defined(ENABLE_BASIC_PRINTING) 1307 #if defined(ENABLE_BASIC_PRINTING)
1303 command_updater_.UpdateCommandEnabled(IDC_BASIC_PRINT, 1308 command_updater_.UpdateCommandEnabled(IDC_BASIC_PRINT,
1304 CanBasicPrint(browser_)); 1309 CanBasicPrint(browser_));
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
1376 1381
1377 BrowserWindow* BrowserCommandController::window() { 1382 BrowserWindow* BrowserCommandController::window() {
1378 return browser_->window(); 1383 return browser_->window();
1379 } 1384 }
1380 1385
1381 Profile* BrowserCommandController::profile() { 1386 Profile* BrowserCommandController::profile() {
1382 return browser_->profile(); 1387 return browser_->profile();
1383 } 1388 }
1384 1389
1385 } // namespace chrome 1390 } // namespace chrome
OLDNEW
« no previous file with comments | « chrome/app/nibs/MainMenu.xib ('k') | chrome/browser/ui/browser_commands_mac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698