Print Preview: Remembering last used printer.

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();
 
 // The total page count of the previewed document regardless of which pages the
 // user has selected.
 var totalPageCount = -1;
 
@@ skipping 12 matching lines @@
 // Used to disable some printing options when the preview is not modifiable.
 var previewModifiable = false;
 
 // Destination list special value constants.
 const PRINT_TO_PDF = 'Print To PDF';
 const MANAGE_PRINTERS = 'Manage Printers';
 
 // State of the print preview settings.
 var printSettings = new PrintSettings();
 
+var defaultOrLastUsedPrinterName = '';
+
 /**
  * Window onload handler, sets up the page and starts print preview by getting
  * the printer list.
  */
 function onLoad() {
   $('system-dialog-link').addEventListener('click', showSystemDialog);
   $('cancel-button').addEventListener('click', handleCancelButtonClick);
 
   if (!checkCompatiblePluginExists()) {
     displayErrorMessage(localStrings.getString('noPlugin'), false);
@@ skipping 305 matching lines @@
 }
 
 /**
  * Set the default printer. If there is one, generate a print preview.
  * @param {string} printer Name of the default printer. Empty if none.
  */
 function setDefaultPrinter(printer) {
   // Add a placeholder value so the printer list looks valid.
   addDestinationListOption('', '', true, true);
   if (printer) {
-    $('printer-list')[0].value = printer;
+    defaultOrLastUsedPrinterName = printer;
+    $('printer-list')[0].value = defaultOrLastUsedPrinterName;
     updateControlsWithSelectedPrinterCapabilities();
   }
   chrome.send('getPrinters');
 }
 
 /**
  * Fill the printer list drop down.
  * Called from PrintPreviewHandler::SendPrinterList().
  * @param {Array} printers Array of printer info objects.
- * @param {number} defaultPrinterIndex The index of the default printer.
  */
-function setPrinters(printers, defaultPrinterIndex) {
+function setPrinters(printers) {

[dpapad, 2011/06/07 01:07:44]

Since we already know the default printer, there is no longer need to include
this info here.

   var printerList = $('printer-list');
   // If there exists a dummy printer value, then setDefaultPrinter() already
   // requested a preview, so no need to do it again.
   var needPreview = (printerList[0].value == '');
   for (var i = 0; i < printers.length; ++i) {
-    // Check if we are looking at the default printer.
-    if (i == defaultPrinterIndex) {
-      // If the default printer from setDefaultPrinter() does not match the
-      // enumerated value, (re)generate the print preview.
-      if (printers[i].deviceName != printerList[0].value)
-        needPreview = true;
-    }
+    var isDefault = (printers[i].deviceName == defaultOrLastUsedPrinterName);
     addDestinationListOption(printers[i].printerName, printers[i].deviceName,
-                             i == defaultPrinterIndex, false);
+                             isDefault, false);
   }
 
   // Remove the dummy printer added in setDefaultPrinter().
   printerList.remove(0);
 
   if (printers.length != 0)
     addDestinationListOption('', '', false, true);
 
   // Adding option for saving PDF to disk.
+  console.log(defaultOrLastUsedPrinterName == PRINT_TO_PDF);
   addDestinationListOption(localStrings.getString('printToPDF'),
-                           PRINT_TO_PDF, false, false);
+                           PRINT_TO_PDF,
+                           defaultOrLastUsedPrinterName == PRINT_TO_PDF,
+                           false);
   addDestinationListOption('', '', false, true);
 
   // Add an option to manage printers.
   addDestinationListOption(localStrings.getString('managePrinters'),
                            MANAGE_PRINTERS, false, false);
 
   printerList.disabled = false;
 
   if (needPreview)
     updateControlsWithSelectedPrinterCapabilities();
@@ skipping 558 matching lines @@
   this.isLandscape = '';
 }
 
 /**
  * Takes a snapshot of the print settings.
  */
 PrintSettings.prototype.save = function() {
   this.deviceName = getSelectedPrinterName();
   this.isLandscape = isLandscape();
 }