Improves application state persistance.

chrome/browser/resources/print_preview/data/destination_store.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';
 
   /**
    * A data store that stores destinations and dispatches events when the data
    * store changes.
    * @param {!print_preview.NativeLayer} nativeLayer Used to fetch local print
    *     destinations.
+   * @param {!print_preview.AppState} appState Application state.
    * @constructor
    * @extends {cr.EventTarget}
    */
-  function DestinationStore(nativeLayer) {
+  function DestinationStore(nativeLayer, appState) {
     cr.EventTarget.call(this);
 
     /**
      * Used to fetch local print destinations.
      * @type {!print_preview.NativeLayer}
      * @private
      */
     this.nativeLayer_ = nativeLayer;
 
     /**
+     * Used to load and persist the selected destination.
+     * @type {!print_preview.AppState}
+     * @private
+     */
+    this.appState_ = appState;
+
+    /**
      * Internal backing store for the data store.
      * @type {!Array.<!print_preview.Destination>}
      * @private
      */
     this.destinations_ = [];
 
     /**
-     * Cache used for constant lookup of destinations.
+     * Cache used for constant lookup of destinations by ID.
      * @type {object.<string, !print_preview.Destination>}
      * @private
      */
     this.destinationMap_ = {};
 
     /**
      * Currently selected destination.
      * @type {print_preview.Destination}
      * @private
      */
@@ skipping 160 matching lines @@
     },
 
     /**
      * @return {boolean} Whether a search for cloud destinations is in progress.
      */
     get isCloudDestinationSearchInProgress() {
       return this.outstandingCloudSearchRequestCount_ > 0;
     },
 
     /**
-     * Sets the initially selected destination. If any inserted destinations
-     * match this ID, that destination will be automatically selected. This
-     * occurs only once for every time this setter is called or if the store is
-     * cleared.
-     * @param {?string} initialDestinationId ID of the destination that should
-     *     be selected automatically when added to the store or {@code null} if
-     *     the first destination that is inserted should be selected.
-     * @param {boolean} isLocalDestination Whether the initial destination is
-     *     local.
+     * Initializes the destination store. Sets the initially selected
+     * destination. If any inserted destinations match this ID, that destination
+     * will be automatically selected. This method must be called after the
+     * print_preview.AppState has been initialized.
+     * @param {?string} systemDefaultDestinationId ID of the system default
+     *     destination.
+     * @private
      */
-    setInitialDestinationId: function(
-        initialDestinationId, isLocalDestination) {
-      this.initialDestinationId_ = initialDestinationId;
-      this.isInitialDestinationLocal_ = isLocalDestination;
+    init: function(systemDefaultDestinationId) {
+      if (this.appState_.selectedDestinationId) {
+        this.initialDestinationId_ = this.appState_.selectedDestinationId;
+        this.isInitialDestinationLocal_ =
+            this.appState_.isSelectedDestinationLocal;
+      } else {
+        this.initialDestinationId_ = systemDefaultDestinationId;
+        this.isInitialDestinationLocal_ = true;
+      }
+
       this.isInAutoSelectMode_ = true;
       if (this.initialDestinationId_ == null) {
         assert(this.destinations_.length > 0,
                'No destinations available to select');
         this.selectDestination(this.destinations_[0]);
       } else {
         var candidate = this.destinationMap_[this.initialDestinationId_];
         if (candidate != null) {
           this.selectDestination(candidate);
-        } else if (!cr.isChromeOS && isLocalDestination) {
+        } else if (!cr.isChromeOS && this.isInitialDestinationLocal_) {
           this.nativeLayer_.startGetLocalDestinationCapabilities(
-              initialDestinationId);
+              this.initialDestinationId_);
         }
       }
     },
 
     /**
      * Sets the destination store's Google Cloud Print interface.
      * @param {!print_preview.CloudPrintInterface} cloudPrintInterface Interface
      *     to set.
      */
     setCloudPrintInterface: function(cloudPrintInterface) {
@@ skipping 31 matching lines @@
       }
       if (destination.id == print_preview.Destination.GooglePromotedId.FEDEX &&
           !destination.isTosAccepted) {
         assert(this.cloudPrintInterface_ != null,
                'Selected FedEx Office destination, but Google Cloud Print is ' +
                'not enabled');
         destination.isTosAccepted = true;
         this.cloudPrintInterface_.updatePrinterTosAcceptance(destination.id,
                                                              true);
       }
+      this.appState_.persistSelectedDestination(this.selectedDestination_);
       cr.dispatchSimpleEvent(
           this, DestinationStore.EventType.DESTINATION_SELECT);
       if (destination.capabilities == null) {
          if (destination.isLocal) {
           this.nativeLayer_.startGetLocalDestinationCapabilities(
               destination.id);
         } else {
           assert(this.cloudPrintInterface_ != null,
                  'Selected destination is a cloud destination, but Google ' +
                  'Cloud Print is not enabled');
@@ skipping 187 matching lines @@
      * @param {cr.Event} event Contains the capabilities of the local print
      *     destination.
      * @private
      */
     onLocalDestinationCapabilitiesSet_: function(event) {
       var destinationId = event.settingsInfo['printerId'];
       var destination = this.destinationMap_[destinationId];
       var capabilities = print_preview.LocalCapabilitiesParser.parse(
             event.settingsInfo);
       if (destination) {
+        // In case there were multiple capabilities request for this local
+        // destination, just ignore the later ones.
+        if (destination.capabilities != null) {
+          return;
+        }
         destination.capabilities = capabilities;
-        if (this.selectedDestination_ &&
-            this.selectedDestination_.id == destinationId) {
-          cr.dispatchSimpleEvent(this,
-                                 DestinationStore.EventType.
-                                     SELECTED_DESTINATION_CAPABILITIES_READY);
-        }
       } else {
         // TODO(rltoscano): This makes the assumption that the "deviceName" is
         // the same as "printerName". We should include the "printerName" in the
         // response. See http://crbug.com/132831.
         destination = print_preview.LocalDestinationParser.parse(
             {deviceName: destinationId, printerName: destinationId});
         destination.capabilities = capabilities;
         this.insertDestination(destination);
       }
+      if (this.selectedDestination_ &&
+          this.selectedDestination_.id == destinationId) {
+        cr.dispatchSimpleEvent(this,
+                               DestinationStore.EventType.
+                                   SELECTED_DESTINATION_CAPABILITIES_READY);
+      }
     },
 
     /**
      * Called when a request to get a local destination's print capabilities
      * fails. If the destination is the initial destination, auto-select another
      * destination instead.
      * @param {cr.Event} event Contains the destination ID that failed.
      * @private
      */
     onGetCapabilitiesFail_: function(event) {
@@ skipping 91 matching lines @@
              'No destinations were loaded before auto-select timeout expired');
       this.selectDestination(this.destinations_[0]);
     }
   };
 
   // Export
   return {
     DestinationStore: DestinationStore
   };
 });