Re-enable Cloud printers for ChromeOS kiosks.

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 567 matching lines @@
     /**
      * Handler for {@code print_preview.DestinationListItem.EventType.
      * CONFIGURE_REQUEST} event, which is called to check a destination list
      * item needs to be setup on Chrome OS before being selected. Note we do not
      * allow configuring more than one destination at the same time.
      * @param {!CustomEvent} event Contains the destination needs to be setup.
      * @private
      */
     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.');
 
       // Another printer setup is in process or the printer doesn't need to be
       // set up. Reject the setup request directly.
       if (this.destinationInConfiguring_ != null ||
           destination.origin != print_preview.Destination.Origin.CROS ||
           destination.capabilities != null) {
         destinationItem.onConfigureRequestRejected(
             this.destinationInConfiguring_ != null);
@@ skipping 229 matching lines @@
     onWindowResize_: function() {
       this.reflowLists_();
     }
   };
 
   // Export
   return {
     DestinationSearch: DestinationSearch
   };
 });