PrintPreview: Make ctrl-shift-p start the native print flow.

chrome/browser/ui/webui/print_preview_handler.cc

 // Copyright (c) 2011 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_handler.h"
 
 #include <string>
 
 #include "base/base64.h"
 #if !defined(OS_CHROMEOS)
@@ skipping 644 matching lines @@
 
 void PrintPreviewHandler::HandleManageCloudPrint(const ListValue*) {
   Browser* browser = BrowserList::GetLastActive();
   browser->OpenURL(CloudPrintURL(browser->profile()).
                    GetCloudPrintServiceManageURL(),
                    GURL(),
                    NEW_FOREGROUND_TAB,
                    PageTransition::LINK);
 }
 
-void PrintPreviewHandler::HandleShowSystemDialog(const ListValue* args) {
+void PrintPreviewHandler::HandleShowSystemDialog(const ListValue*) {

[Lei Zhang, 2011/08/24 01:27:28]

This is just going to conflict with my cleanup CL.

[kmadhusu, 2011/08/25 02:15:20]

okay :)

   ReportStats();
   ReportUserActionHistogram(FALLBACK_TO_ADVANCED_SETTINGS_DIALOG);
 
   TabContents* initiator_tab = GetInitiatorTab();
   if (!initiator_tab)
     return;
 
   TabContentsWrapper* wrapper =
       TabContentsWrapper::GetCurrentWrapperForContents(initiator_tab);
   printing::PrintViewManager* manager = wrapper->print_view_manager();
   manager->set_observer(this);
   manager->PrintForSystemDialogNow();
+
+  // Cancel the pending preview request if exists.
+  PrintPreviewUI* print_preview_ui = static_cast<PrintPreviewUI*>(web_ui_);
+  print_preview_ui->OnPrintPreviewRequest(-1);

[Lei Zhang, 2011/08/24 01:27:28]

Let's add a new method to PrintPreviewUI to do cancellations.

[kmadhusu, 2011/08/25 02:15:20]

Done.

 }
 
 void PrintPreviewHandler::HandleManagePrinters(const ListValue* args) {
   ++manage_printers_dialog_request_count_;
   printing::PrinterManagerDialog::ShowPrinterManagerDialog();
 }
 
 void PrintPreviewHandler::HandleReloadCrashedInitiatorTab(const ListValue*) {
   ReportStats();
   ReportUserActionHistogram(INITIATOR_TAB_CRASHED);
@@ skipping 238 matching lines @@
     return;
 
   // We no longer require the initiator tab details. Remove those details
   // associated with the preview tab to allow the initiator tab to create
   // another preview tab.
   printing::PrintPreviewTabController* tab_controller =
      printing::PrintPreviewTabController::GetInstance();
   if (tab_controller)
     tab_controller->EraseInitiatorTabInfo(preview_tab());
 }