[CUPS] Implement the enterprise icon for printers in Print Preview Dialog.

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

Index: chrome/browser/resources/print_preview/data/destination.js
diff --git a/chrome/browser/resources/print_preview/data/destination.js b/chrome/browser/resources/print_preview/data/destination.js
index c4ae78e4db36e9b98176fb370d8bdff28a489f99..d5db22331b81e341799dbe9936bb80738c3bf580 100644
--- a/chrome/browser/resources/print_preview/data/destination.js
+++ b/chrome/browser/resources/print_preview/data/destination.js
@@ -261,6 +261,15 @@ cr.define('print_preview', function() {
   };
 
   /**
+   * Special tag that denotes whether the destination is an enterprise policy
+   * controlled printer.
+   * @type {string}
+   * @const
+   * @private
+   */
+  Destination.EnterprisePrinterTag_ = '__cp__cups_enterprise_printer=true';
+
+  /**
    * Enumeration of relative icon URLs for various types of destinations.
    * @enum {string}
    * @private
@@ -274,7 +283,8 @@ cr.define('print_preview', function() {
     THIRD_PARTY: 'images/third_party.png',
     PDF: 'images/pdf.png',
     DOCS: 'images/google_doc.png',
-    FEDEX: 'images/third_party_fedex.png'
+    FEDEX: 'images/third_party_fedex.png',
+    ENTERPRISE: 'images/business.svg'
   };
 
   Destination.prototype = {
@@ -489,6 +499,9 @@ cr.define('print_preview', function() {
       if (this.id_ == Destination.GooglePromotedId.SAVE_AS_PDF) {
         return Destination.IconUrl_.PDF;
       }
+      if (this.isEnterprisePrinter) {
+        return Destination.IconUrl_.ENTERPRISE;
+      }
       if (this.isLocal) {
         return Destination.IconUrl_.LOCAL;
       }
@@ -558,7 +571,15 @@ cr.define('print_preview', function() {
      */
     get isProvisional() {
       return this.provisionalType_ != Destination.ProvisionalType.NONE;
-    }
+    },
+
+    /**
+     * Whether the printer is enterprise policy controlled printer.
+     * @return {boolean}
+     */
+    get isEnterprisePrinter() {
+      return arrayContains(this.tags_, Destination.EnterprisePrinterTag_);

[dpapad, 2017/01/18 02:56:04]

Is this linear search going to be costly? What is the size of |tags_| and how
often is this function called?

[xdai1, 2017/01/18 18:45:42]

For a single printer, the size of |tags_| equals to the number of the cups
printer's options
(https://cs.chromium.org/chromium/src/build/linux/debian_wheezy_amd64-sysroot/...).
But I don't know how many options a cups printer can have (maybe skau@ knows?) .
This function is called when we initialize the printers list and when the
printer state changes.

[skau, 2017/01/18 20:59:06]

FYI: We expect around 20 printer properties for any given printer.  It could be
more but won't exceed 100.

[dpapad, 2017/01/18 21:09:57]

So if I have hundreds of printers (like in a Google office), is this going to be
a problem? In other words, is there a noticeable slow down in startup (when the
printer list gets initialized)? Can/should |this.tags_| be converted to a native
Set instead, see
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Obje...

[skau, 2017/01/18 22:05:40]

xdai@ We should probably make isEnterprise a separate field on the destination. 
Looks like you could extract the value from the options map in
print_preview_handler when converting from a PrintBasicInfo to the dictionary.

[xdai1, 2017/01/19 01:10:17]

I made isEnterprise_ a separate field on the destination object as you
suggested. PrintersToValues() has already extracted the value and saved it in
the dictionary with the key printing::kSettingPrinterOptions (printerOptions).

[xdai1, 2017/01/19 01:10:17]

the format of|this.tags_|
(https://cs.chromium.org/chromium/src/chrome/browser/resources/print_preview/d...)
is defined to be consistent with cloud print tags format. So it might be better
to leave the format as is for now...

+    },
   };
 
   // Export