Perform printer setup on Chrome OS before selecting printer.

chrome/browser/resources/print_preview/search/destination_search.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';
 
   /**
    * Component used for searching for a print destination.
    * This is a modal dialog that allows the user to search and select a
@@ skipping 565 matching lines @@
     },
 
     /**
      * Called when a destination is selected. Clears the search and hides the
      * widget. If The destination is provisional, it runs provisional
      * destination resolver first.
      * @param {!print_preview.Destination} destination The selected destination.
      * @private
      */
     handleOnDestinationSelect_: function(destination) {
+      if (destination.origin == print_preview.Destination.Origin.CROS &&
+          !destination.capabilities) {
+        // local printers on CrOS require setup.
+        assert(!this.printerConfigurer_);
+        this.printerConfigurer_ = new print_preview.CrosDestinationResolver(
+            this.destinationStore_, destination);
+        this.addChild(this.printerConfigurer_);
+        this.printerConfigurer_.run(this.getElement())
+            .then(

[dpapad, 2017/01/11 20:36:31]

From JS styleguide "Always put the operator on the preceding line." and "This
includes the dot operator.",
https://google.github.io/styleguide/javascriptguide.xml.

[skau, 2017/01/12 00:36:16]

Weird.  I never noticed that.  I've fixed the similar block below for
consistency.

+                /**
+                 * @param {Object} result
+                 *    An object containing the printerId and capabilities.
+                 */
+                function(result) {
+                  assert(result.printerId == destination.id);
+                  destination.capabilities = result.capabilities;
+                  this.handleOnDestinationSelect_(destination);
+                }.bind(this))
+            .catch(
+                function() {
+                  console.log('Failed to setup destination: ' + destination.id);

[dpapad, 2017/01/11 20:36:31]

console.error?

[skau, 2017/01/12 00:36:16]

I've made it a warning.  We'll want to know if it failed since we don't expect
it to fail. However, the dialog will still be usable in the event that the call
does fail so it's not an error.  More importantly, if it's an error, the unit
test won't pass.

+                })
+            .then(
+                function() {
+                  this.removeChild(this.printerConfigurer_);
+                  this.printerConfigurer_ = null;
+                }.bind(this));
+        return;
+      }
+
       if (destination.isProvisional) {
         assert(!this.provisionalDestinationResolver_,
                'Provisional destination resolver already exists.');
         this.provisionalDestinationResolver_ =
             print_preview.ProvisionalDestinationResolver.create(
                 this.destinationStore_, destination);
         assert(!!this.provisionalDestinationResolver_,
                'Unable to create provisional destination resolver');
 
         var lastFocusedElement = document.activeElement;
@@ skipping 168 matching lines @@
     onWindowResize_: function() {
       this.reflowLists_();
     }
   };
 
   // Export
   return {
     DestinationSearch: DestinationSearch
   };
 });