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 654 matching lines @@
 
   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->OnCancelPendingPreviewRequest();

[Lei Zhang, 2011/08/25 09:05:49]

You can pull this, the associated PrintPreviewUI changes, and the JS change into
its own CL and get it committed sooner.

[kmadhusu, 2011/08/25 17:51:48]

Done.

 }
 
 void PrintPreviewHandler::HandleManagePrinters(const ListValue*) {
   ++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());
 }