Fixes cloud-printer being treated as local-printer problem.

chrome/browser/resources/print_preview/native_layer.js

 // Copyright (c) 2012 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.
 
 cr.define('print_preview', function() {
   'use strict';
 
   /**
    * An interface to the native Chromium printing system layer.
    * @constructor
@@ skipping 36 matching lines @@
    * @const
    */
   NativeLayer.EventType = {
     CAPABILITIES_SET: 'print_preview.NativeLayer.CAPABILITIES_SET',
     CLOUD_PRINT_ENABLE: 'print_preview.NativeLayer.CLOUD_PRINT_ENABLE',
     DESTINATIONS_RELOAD: 'print_preview.NativeLayer.DESTINATIONS_RELOAD',
     DISABLE_SCALING: 'print_preview.NativeLayer.DISABLE_SCALING',
     FILE_SELECTION_CANCEL: 'print_preview.NativeLayer.FILE_SELECTION_CANCEL',
     FILE_SELECTION_COMPLETE:
         'print_preview.NativeLayer.FILE_SELECTION_COMPLETE',
+    GET_CAPABILITIES_FAIL: 'print_preview.NativeLayer.GET_CAPABILITIES_FAIL',
     INITIAL_SETTINGS_SET: 'print_preview.NativeLayer.INITIAL_SETTINGS_SET',
     LOCAL_DESTINATIONS_SET: 'print_preview.NativeLayer.LOCAL_DESTINATIONS_SET',
     PAGE_COUNT_READY: 'print_preview.NativeLayer.PAGE_COUNT_READY',
     PAGE_LAYOUT_READY: 'print_preview.NativeLayer.PAGE_LAYOUT_READY',
     PAGE_PREVIEW_READY: 'print_preview.NativeLayer.PAGE_PREVIEW_READY',
     PREVIEW_GENERATION_DONE:
         'print_preview.NativeLayer.PREVIEW_GENERATION_DONE',
     PREVIEW_GENERATION_FAIL:
         'print_preview.NativeLayer.PREVIEW_GENERATION_FAIL',
     PREVIEW_RELOAD: 'print_preview.NativeLayer.PREVIEW_RELOAD',
@@ skipping 14 matching lines @@
   /**
    * Enumeration of color modes used by Chromium.
    * @enum {number}
    * @private
    */
   NativeLayer.ColorMode_ = {
     GRAY: 1,
     COLOR: 2
   };
 
+  /**
+   * Version of the serialized state of the print preview.
+   * @type {number}
+   * @const
+   * @private
+   */
+  NativeLayer.SERIALIZED_STATE_VERSION_ = 1;
+
   NativeLayer.prototype = {
     __proto__: cr.EventTarget.prototype,
 
     /** Gets the initial settings to initialize the print preview with. */
     startGetInitialSettings: function() {
       chrome.send('getInitialSettings');
     },
 
     /**
      * Requests the system's local print destinations. A LOCAL_DESTINATIONS_SET
@@ skipping 88 matching lines @@
     },
 
     /**
      * Persists the selected destination and print ticket for the next print
      * session.
      * @param {!print_preview.Destination} destination Destination to save.
      * @param {!print_preview.PrintTicketStore} printTicketStore Used for
      *     generating the serialized print ticket to persist.
      */
     startSaveDestinationAndTicket: function(destination, printTicketStore) {
-      chrome.send('saveLastPrinter', [destination.id, '' /*TODO(rltoscano)*/]);
+      // TODO(rltoscano): Implement a comprehensive serialization process.
+      var printPreviewSerializedState = {
+        'version': print_preview.NativeLayer.SERIALIZED_STATE_VERSION_,
+        'isLocalDestination':
+            destination.type == print_preview.Destination.Type.LOCAL
+      };
+      chrome.send(
+          'saveLastPrinter',
+          [destination.id, JSON.stringify(printPreviewSerializedState)]);
     },
 
     /**
      * Requests that the document be printed.
      * @param {!print_preview.Destination} destination Destination to print to.
      * @param {!print_preview.PrintTicketStore} printTicketStore Used to get the
      *     state of the print ticket.
      * @param {print_preview.CloudPrintInterface} cloudPrintInterface Interface
      *     to Google Cloud Print.
      * @param {boolean=} opt_isOpenPdfInPreview Whether to open the PDF in the
@@ skipping 137 matching lines @@
             initialSettings['marginRight'] || 0,
             initialSettings['marginBottom'] || 0,
             initialSettings['marginLeft'] || 0);
       }
 
       var marginsType = null;
       if (initialSettings.hasOwnProperty('marginsType')) {
         marginsType = initialSettings['marginsType'];
       }
 
+      // TODO(rltoscano): Replace this loading of serialized state with a
+      // comprehensive version.
+      var serializedPrintPreviewState =
+          JSON.parse(initialSettings['cloudPrintData'] || '{}');
+      var isLocalDestination = true;
+      if (serializedPrintPreviewState &&
+          serializedPrintPreviewState.version == 1) {
+        isLocalDestination = serializedPrintPreviewState.isLocalDestination;
+      }
+
       var nativeInitialSettings = new print_preview.NativeInitialSettings(
           initialSettings['printAutomaticallyInKioskMode'] || false,
           numberFormatSymbols[0] || ',',
           numberFormatSymbols[1] || '.',
           unitType,
           initialSettings['previewModifiable'] || false,
           initialSettings['initiatorTabTitle'] || '',
           marginsType,
           customMargins,
           initialSettings['duplex'] || false,
           initialSettings['headerFooterEnabled'] || false,
-          initialSettings['printerName'] || null);
+          initialSettings['printerName'] || null,
+          isLocalDestination);
 
       var initialSettingsSetEvent = new cr.Event(
           NativeLayer.EventType.INITIAL_SETTINGS_SET);
       initialSettingsSetEvent.initialSettings = nativeInitialSettings;
       this.dispatchEvent(initialSettingsSetEvent);
     },
 
     /**
      * Turn on the integration of Cloud Print.
      * @param {string} cloudPrintURL The URL to use for cloud print servers.
@@ skipping 27 matching lines @@
      */
     onUpdateWithPrinterCapabilities_: function(settingsInfo) {
       var capsSetEvent = new cr.Event(NativeLayer.EventType.CAPABILITIES_SET);
       capsSetEvent.settingsInfo = settingsInfo;
       this.dispatchEvent(capsSetEvent);
     },
 
     /**
      * Called when native layer gets settings information for a requested local
      * destination.
-     * @param {Object} printer_name printer affected by error.
+     * @param {string} printerId printer affected by error.
      * @private
      */
-    onFailedToGetPrinterCapabilities_: function(printer_name) {
-      // TODO(rltoscano): Switch to the next printer.
-      console.log('onFailedToGetPrinterCapabilities: ' + printer_name);
+    onFailedToGetPrinterCapabilities_: function(destinationId) {
+      var getCapsFailEvent = new cr.Event(
+          NativeLayer.EventType.GET_CAPABILITIES_FAIL);
+      getCapsFailEvent.destinationId = destinationId;
+      this.dispatchEvent(getCapsFailEvent);
     },
 
     /** Reloads the printer list. */
     onReloadPrintersList_: function() {
       cr.dispatchSimpleEvent(this, NativeLayer.EventType.DESTINATIONS_RELOAD);
     },
 
     /**
      * Called from the C++ layer.
      * Take the PDF data handed to us and submit it to the cloud, closing the
@@ skipping 158 matching lines @@
    *     modifiable.
    * @param {string} documentTitle Title of the document.
    * @param {print_preview.ticket_items.MarginsType.Value} marginsType Initial
    *     margins type.
    * @param {print_preview.Margins} customMargins Initial custom margins.
    * @param {boolean} isDuplexEnabled Whether duplexing is initially enabled.
    * @param {boolean} isHeaderFooterEnabled Whether the header-footer is
    *     initially enabled.
    * @param {?string} initialDestinationId ID of the destination to initially
    *     select.
+   * @param {boolean} isLocalDestination Whether the initial destination is
+   *     local.
    * @constructor
    */
   function NativeInitialSettings(
       isInKioskAutoPrintMode,
       thousandsDelimeter,
       decimalDelimeter,
       unitType,
       isDocumentModifiable,
       documentTitle,
       marginsType,
       customMargins,
       isDuplexEnabled,
       isHeaderFooterEnabled,
-      initialDestinationId) {
+      initialDestinationId,
+      isLocalDestination) {
 
     /**
      * Whether the print preview should be in auto-print mode.
      * @type {boolean}
      * @private
      */
     this.isInKioskAutoPrintMode_ = isInKioskAutoPrintMode;
 
     /**
      * Character delimeter of thousands digits.
@@ skipping 57 matching lines @@
      * @private
      */
     this.isHeaderFooterEnabled_ = isHeaderFooterEnabled;
 
     /**
      * ID of the initially selected destination.
      * @type {?string}
      * @private
      */
     this.initialDestinationId_ = initialDestinationId;
+
+    /**
+     * Whether the initial destination is local.
+     * @type {boolean}
+     * @private
+     */
+    this.isLocalDestination_ = isLocalDestination;
   };
 
   NativeInitialSettings.prototype = {
     /**
      * @return {boolean} Whether the print preview should be in auto-print mode.
      */
     get isInKioskAutoPrintMode() {
       return this.isInKioskAutoPrintMode_;
     },
 
@@ skipping 44 matching lines @@
     },
 
     /** @return {boolean} Whether the header-footer is initially enabled. */
     get isHeaderFooterEnabled() {
       return this.isHeaderFooterEnabled_;
     },
 
     /** @return {?string} ID of the initially selected destination. */
     get initialDestinationId() {
       return this.initialDestinationId_;
+    },
+
+    /** @return {boolean} Whether the initial destination is local. */
+    get isLocalDestination() {
+      return this.isLocalDestination_;
     }
   };
 
   // Export
   return {
     NativeInitialSettings: NativeInitialSettings,
     NativeLayer: NativeLayer
   };
 });