Print Preview: Provide a link to try printing with the system dialog when print preview fails. Al...

chrome/browser/resources/print_preview/print_preview.js

 // 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.
 
 // require: cr/ui/print_preview_cloud.js
 
 var localStrings = new LocalStrings();
 
 // If useCloudPrint is true we attempt to connect to cloud print
 // and populate the list of printers with cloud print printers.
@@ skipping 786 matching lines @@
   $('system-dialog-throbber').classList.add('hidden');
   $('native-print-dialog-throbber').classList.add('hidden');
   displayErrorMessage(errorMessage);
 }
 
 /**
  * Display an error message when print preview fails.
  * Called from PrintPreviewMessageHandler::OnPrintPreviewFailed().
  */
 function printPreviewFailed() {
-  displayErrorMessage(localStrings.getString('previewFailed'));
+  displayErrorMessageWithButton(localStrings.getString('previewFailed'),
+                                localStrings.getString('launchNativeDialog'),
+                                launchNativePrintDialog);

[kmadhusu, 2011/08/24 23:30:43]

I thought launchNativePrintDialog is for cloud print UI and
onSystemDialogLinkClicked is for our system dialog. Is there any specific reason
to call launchNativePrintDialog?

[Lei Zhang, 2011/08/24 23:46:25]

launchNativePrintDialog is for chromium, not cloud print. As in, chromium builds
with print preview enabled, but no pdf plugin.

 }
 
 /**
  * Called when the PDF plugin loads its document.
  */
 function onPDFLoad() {
   if (previewModifiable) {
     setPluginPreviewPageCount();
   }
   $('pdf-viewer').fitToHeight();
@@ skipping 162 matching lines @@
 <include src="print_preview_animations.js"/>
 <include src="print_preview_cloud.js"/>
 <include src="print_preview_utils.js"/>
 <include src="print_header.js"/>
 <include src="page_settings.js"/>
 <include src="copies_settings.js"/>
 <include src="header_footer_settings.js"/>
 <include src="layout_settings.js"/>
 <include src="color_settings.js"/>
 <include src="margin_settings.js"/>