Introduce concept of provisional destinations to print preview

chrome/browser/resources/print_preview/data/destination_store.js

Index: chrome/browser/resources/print_preview/data/destination_store.js
diff --git a/chrome/browser/resources/print_preview/data/destination_store.js b/chrome/browser/resources/print_preview/data/destination_store.js
index 3ed368e622456ef107d5a416901033157ba57a55..bbbddce68adb25965313af0cd9fef11dec42a74e 100644
--- a/chrome/browser/resources/print_preview/data/destination_store.js
+++ b/chrome/browser/resources/print_preview/data/destination_store.js
@@ -199,6 +199,8 @@ cr.define('print_preview', function() {
     DESTINATION_SELECT: 'print_preview.DestinationStore.DESTINATION_SELECT',
     DESTINATIONS_INSERTED:
         'print_preview.DestinationStore.DESTINATIONS_INSERTED',
+    PROVISIONAL_DESTINATION_RESOLVED:
+        'print_preview.DestinationStore.PROVISIONAL_DESTINATION_RESOLVED',
     CACHED_SELECTED_DESTINATION_INFO_READY:
         'print_preview.DestinationStore.CACHED_SELECTED_DESTINATION_INFO_READY',
     SELECTED_DESTINATION_CAPABILITIES_READY:
@@ -461,6 +463,10 @@ cr.define('print_preview', function() {
             this, DestinationStore.EventType.DESTINATION_SELECT);
         return;
       }
+
+      assert(!destination.isProvisional,
+             'Unable to select provisonal destinations');
+
       // Update and persist selected destination.
       this.selectedDestination_ = destination;
       this.selectedDestination_.isRecent = true;
@@ -512,6 +518,19 @@ cr.define('print_preview', function() {
     },
 
     /**
+     * Attempts to resolve a provisional destination.
+     * @param {!print_preview.Destination} destinaion Provisional destination
+     *     that should be resolved.
+     */
+    resolveProvisionalDestination: function(destination) {
+      assert(
+          destination.provisionalType ==
+              print_preview.Destination.ProvisionalType.NEEDS_USB_PERMISSION,
+          'Provisional type cannot be resolved.');
+      this.nativeLayer_.grantExtensionPrinterAccess(destination.id);
+    },
+
+    /**
      * Selects 'Save to PDF' destination (since it always exists).
      * @private
      */
@@ -614,6 +633,53 @@ cr.define('print_preview', function() {
     },
 
     /**
+     * Event handler for {@code
+     * print_preview.NativeLayer.EventType.PROVISIONAL_DESTINATION_RESOLVED}.
+     * Currently assumes the provisional destination is an extension
+     * destination.
+     * Called when a provisional destination resolvement attempt finishes.
+     * The provisional destination is removed from the store and replaced with
+     * a destination created from the resolved destination properties, if any
+     * are reported.
+     * Emits {@code DestinationStore.EventType.PROVISIONAL_DESTINATION_RESOLVED}
+     * event.
+     * @param {!Event} The event containing the provisional destination ID and
+     *     resolved destination description. If the destination was not
+     *     successfully resolved, the description will not be set.
+     * @private
+     */
+    handleProvisionalDestinationResolved_: function(evt) {
+      var provisionalDestinationIndex = -1;
+      var provisionalDestination = null;
+      for (var i = 0; i < this.destinations_.length; ++i) {
+        if (evt.provisionalId == this.destinations_[i].id) {
+          provisionalDestinationIndex = i;
+          provisionalDestination = this.destinations_[i];
+          break;
+        }
+      }
+
+      if (!provisionalDestination)
+        return;
+
+      this.destinations_.splice(provisionalDestinationIndex, 1);
+      this.destinationMap_[this.getKey_(provisionalDestination)] = undefined;

[Aleksey Shlyapnikov, 2015/06/08 23:00:54]

delete this.destinationMap_[this.getKey_(provisionalDestination)];

[tbarzic, 2015/06/09 18:21:59]

Done.

+
+      var destination = evt.destination ?
+          print_preview.ExtensionDestinationParser.parse(evt.destination) :
+          null;
+
+      if (destination)
+        this.insertIntoStore_(destination);
+
+      var event = new Event(
+          DestinationStore.EventType.PROVISIONAL_DESTINATION_RESOLVED);
+      event.provisionalId = evt.provisionalId;
+      event.destination = destination;
+      this.dispatchEvent(event);
+    },
+
+    /**
      * Inserts {@code destination} to the data store and dispatches a
      * DESTINATIONS_INSERTED event.
      * @param {!print_preview.Destination} destination Print destination to
@@ -782,6 +848,10 @@ cr.define('print_preview', function() {
           this.nativeLayer_,
           print_preview.NativeLayer.EventType.EXTENSION_CAPABILITIES_SET,
           this.onExtensionCapabilitiesSet_.bind(this));
+      this.tracker_.add(
+          this.nativeLayer_,
+          print_preview.NativeLayer.EventType.PROVISIONAL_DESTINATION_RESOLVED,
+          this.handleProvisionalDestinationResolved_.bind(this));
     },
 
     /**