PrintPreview:[MAC & WIN] Added code to show the native printer management dialog.

chrome/browser/resources/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.
 
 var localStrings = new LocalStrings();
 
 // Whether or not the PDF plugin supports all the capabilities needed for
 // print preview.
 var hasCompatiblePDFPlugin = true;
 
 // The total page count of the previewed document regardless of which pages the
 // user has selected.
 var totalPageCount = -1;
 
 // The previously selected pages by the user. It is used in
 // onPageSelectionMayHaveChanged() to make sure that a new preview is not
 // requested more often than necessary.
 var previouslySelectedPages = [];
 
 // The previously selected layout mode. It is used in order to prevent the
 // preview from updating when the user clicks on the already selected layout
 // mode.
 var previouslySelectedLayout = null;
 
 // Timer id of the page range textfield. It is used to reset the timer whenever
 // needed.
 var timerId;
 
+// Store the last selected printer index.
+var lastSelectedPrinterIndex = 0;
+
 /**
  * Window onload handler, sets up the page and starts print preview by getting
  * the printer list.
  */
 function onLoad() {
   initializeAnimation();
 
   $('printer-list').disabled = true;
   $('print-button').disabled = true;
   $('print-button').addEventListener('click', printFile);
@@ skipping 50 matching lines @@
 function updateControlsWithSelectedPrinterCapabilities() {
   var printerList = $('printer-list');
   var selectedPrinter = printerList.selectedIndex;
   if (selectedPrinter < 0)
     return;
 
   var printerName = printerList.options[selectedPrinter].textContent;
   if (printerName == localStrings.getString('printToPDF')) {
     updateWithPrinterCapabilities({'disableColorOption': true,
                                    'setColorAsDefault': true});
+  } else if (printerName == localStrings.getString('managePrinters')) {
+    printerList.selectedIndex = lastSelectedPrinterIndex;
+    chrome.send('managePrinters');
+    return;
   } else {
     // This message will call back to 'updateWithPrinterCapabilities'
     // function.
     chrome.send('getPrinterCapabilities', [printerName]);
   }
+
+  if (lastSelectedPrinterIndex != selectedPrinter)

[Lei Zhang, 2011/04/27 23:46:36]

no need for this if statement.

[kmadhusu, 2011/04/28 00:08:26]

Done.

+    lastSelectedPrinterIndex = selectedPrinter;
 }
 
 /**
  * Updates the controls with printer capabilities information.
  * @param {Object} settingInfo printer setting information.
  */
 function updateWithPrinterCapabilities(settingInfo) {
   var disableColorOption = settingInfo.disableColorOption;
   var setColorAsDefault = settingInfo.setColorAsDefault;
   var colorOption = $('color');
@@ skipping 207 matching lines @@
 }
 
 /**
  * Fill the printer list drop down.
  * Called from PrintPreviewHandler::SendPrinterList().
  * @param {Array} printers Array of printer names.
  * @param {number} defaultPrinterIndex The index of the default printer.
  */
 function setPrinters(printers, defaultPrinterIndex) {
   var printerList = $('printer-list');
-  for (var i = 0; i < printers.length; ++i) {
-    var option = document.createElement('option');
-    option.textContent = printers[i];
-    printerList.add(option);
-    if (i == defaultPrinterIndex)
-      option.selected = true;
-  }
+  for (var i = 0; i < printers.length; ++i)
+    addDestinationListOption(printers[i], i == defaultPrinterIndex);
 
   // Adding option for saving PDF to disk.
-  var option = document.createElement('option');
-  option.textContent = localStrings.getString('printToPDF');
-  printerList.add(option);
+  addDestinationListOption(localStrings.getString('printToPDF'), false);
+
+  // Add an option to manage printers.
+  addDestinationListOption(localStrings.getString('managePrinters'), false);

[Lei Zhang, 2011/04/27 23:46:36]

nit: can you keep the new line after this?

[kmadhusu, 2011/04/28 00:08:26]

Done.

   printerList.disabled = false;
 
   updateControlsWithSelectedPrinterCapabilities();
 
   // Once the printer list is populated, generate the initial preview.
   requestPrintPreview();
 }
 
 /**
+ * Adds an option to the destination list.

[Lei Zhang, 2011/04/27 23:46:36]

nit: the printer destination list.

[kmadhusu, 2011/04/28 00:08:26]

Done.

+ * @param {String} optionText specifies the option text content.
+ * @param {boolean} is_default is true if the option needs to be selected.
+ */
+function addDestinationListOption(optionText, is_default) {
+  var option = document.createElement('option');
+  option.textContent = optionText;
+  $('printer-list').add(option);
+  if (is_default)
+    option.selected = true;
+}
+
+/**
  * Sets the color mode for the PDF plugin.
  * Called from PrintPreviewHandler::ProcessColorSetting().
  * @param {boolean} color is true if the PDF plugin should display in color.
  */
 function setColor(color) {
   if (!hasCompatiblePDFPlugin) {
     return;
   }
   var pdfViewer = $('pdf-viewer');
   if (!pdfViewer) {
@@ skipping 355 matching lines @@
  * Returns true if the contents of the two arrays are equal.
  */
 function areArraysEqual(array1, array2) {
   if (array1.length != array2.length)
     return false;
   for (var i = 0; i < array1.length; i++)
     if(array1[i] != array2[i])
       return false;
   return true;
 }