Re-enable Cloud printers for ChromeOS kiosks.

chrome/browser/resources/print_preview/search/destination_search.js

Index: chrome/browser/resources/print_preview/search/destination_search.js
diff --git a/chrome/browser/resources/print_preview/search/destination_search.js b/chrome/browser/resources/print_preview/search/destination_search.js
index c9531b014aeff7d0d37cc3891d3c9e10a5e80b7c..58e0a12d3ead98dbc60f04d24598633d643e9b89 100644
--- a/chrome/browser/resources/print_preview/search/destination_search.js
+++ b/chrome/browser/resources/print_preview/search/destination_search.js
@@ -585,9 +585,15 @@ cr.define('print_preview', function() {
      */
     onDestinationConfigureRequest_: function(event) {
       var destination = event.detail.destination;
-      var destinationItem = destination.isLocal ?
-          this.localList_.getDestinationItem(destination.id) :
-          this.cloudList_.getDestinationItem(destination.id);
+      var destinationItem;

[Lei Zhang, 2017/04/25 00:42:36]

Looking at renderDestinations_() above, should this block instead be the
following?

var destinationItem =
    destination.isLocal ||
    destination.origin == print_preview.Destination.Origin.DEVICE ?
        this.localList_.getDestinationItem(destination.id) :
        this.cloudList_.getDestinationItem(destination.id);

See:
https://crrev.com/296501
https://crrev.com/301675
https://crbug.com/416701 <-- Why is this assigned to me?!

[skau, 2017/04/25 00:58:05]

Hmm.  Looks like somebody needs to think about how printer types should be
handled in the UI...

But in the meantime, I've narrowed the condition to just Origin.DEVICE printers.

+      if (!destination.isLocal) {
+        destinationItem = this.cloudList_.getDestinationItem(destination.id);
+      }
+      // Non-local destinations can be in the local list for Cloud Printers
+      // in Kiosk apps. crbug.com/713831
+      if (destinationItem == null) {
+        destinationItem = this.localList_.getDestinationItem(destination.id);
+      }
       assert(destinationItem != null,
             'User does not select a valid destination item.');