Disable Ctrl+Shift+P on Chrome OS

chrome/browser/ui/webui/print_preview/print_preview_handler.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/print_preview/print_preview_handler.h"
 
 #include <ctype.h>
 #include <stddef.h>
 
 #include <map>
@@ skipping 1127 matching lines @@
   if (!args->GetString(0, &user))
     return;
   if (!user.empty())
     manage_url = net::AppendQueryParameter(manage_url, "user", user);
   preview_web_contents()->OpenURL(
       content::OpenURLParams(manage_url, content::Referrer(),
                              WindowOpenDisposition::NEW_FOREGROUND_TAB,
                              ui::PAGE_TRANSITION_LINK, false));
 }
 
 #if defined(ENABLE_BASIC_PRINTING)

[Lei Zhang, 2016/10/03 18:33:03]

Why not just change this so HandleShowSystemDialog() only exists in the relevant
build config?

[skau, 2016/10/03 21:52:05]

Done.

 void PrintPreviewHandler::HandleShowSystemDialog(
     const base::ListValue* /*args*/) {
+#if defined(OS_CHROMEOS)
+  NOTREACHED();
+#else
   ReportStats();
   ReportUserActionHistogram(FALLBACK_TO_ADVANCED_SETTINGS_DIALOG);
 
   WebContents* initiator = GetInitiator();
   if (!initiator)
     return;
 
   printing::PrintViewManager* print_view_manager =
       printing::PrintViewManager::FromWebContents(initiator);
   print_view_manager->PrintForSystemDialogNow(
       base::Bind(&PrintPreviewHandler::ClosePreviewDialog,
                  weak_factory_.GetWeakPtr()));
 
   // Cancel the pending preview request if exists.
   print_preview_ui()->OnCancelPendingPreviewRequest();
+#endif  // OS_CHROMEOS
 }
+
 #endif  // ENABLE_BASIC_PRINTING
 
 void PrintPreviewHandler::HandleManagePrinters(
     const base::ListValue* /*args*/) {
   ++manage_printers_dialog_request_count_;
 #if defined(OS_CHROMEOS)
   GURL local_printers_manage_url(chrome::kChromeUIMdCupsSettingsURL);
   preview_web_contents()->OpenURL(
       content::OpenURLParams(local_printers_manage_url, content::Referrer(),
                              WindowOpenDisposition::NEW_FOREGROUND_TAB,
@@ skipping 589 matching lines @@
 
 void PrintPreviewHandler::UnregisterForGaiaCookieChanges() {
   if (gaia_cookie_manager_service_)
     gaia_cookie_manager_service_->RemoveObserver(this);
 }
 
 void PrintPreviewHandler::SetPdfSavedClosureForTesting(
     const base::Closure& closure) {
   pdf_file_saved_closure_ = closure;
 }