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

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

Issue 522493002: Removed Ctrl+Shift+P shortcut for system dialog printing on Windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Thu Aug 28 21:07:58 PDT 2014 Created 6 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
« no previous file with comments | « chrome/app/chrome_dll.rc ('k') | chrome/browser/ui/views/accelerator_table.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 cc041b4646a4f966599c5f6fecab6f29bc966226..2adf3760692aa7a9dc36b7f019a170b1267d5283 100644
--- a/chrome/browser/ui/browser_command_controller.cc
+++ b/chrome/browser/ui/browser_command_controller.cc
@@ -542,10 +542,12 @@ void BrowserCommandController::ExecuteCommandWithDisposition(
case IDC_PRINT:
Print(browser_);
break;
+#if !defined(OS_WIN)
case IDC_ADVANCED_PRINT:
content::RecordAction(base::UserMetricsAction("Accel_Advanced_Print"));
AdvancedPrint(browser_);
break;
+#endif // OS_WIN
case IDC_PRINT_TO_DESTINATION:
PrintToDestination(browser_);
break;
@@ -1273,8 +1275,10 @@ void BrowserCommandController::UpdateCommandsForFullscreenMode() {
void BrowserCommandController::UpdatePrintingState() {
bool print_enabled = CanPrint(browser_);
command_updater_.UpdateCommandEnabled(IDC_PRINT, print_enabled);
+#if !defined(OS_WIN)
command_updater_.UpdateCommandEnabled(IDC_ADVANCED_PRINT,
CanAdvancedPrint(browser_));
Lei Zhang 2014/08/29 04:30:41 Also ifdef out CanAdvancedPrint() and AdvancedPrin
Vitaly Buka (NO REVIEWS) 2014/08/29 08:06:23 Done.
+#endif
command_updater_.UpdateCommandEnabled(IDC_PRINT_TO_DESTINATION,
print_enabled);
#if defined(OS_WIN)
« no previous file with comments | « chrome/app/chrome_dll.rc ('k') | chrome/browser/ui/views/accelerator_table.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698