Print Preview: Alter passing of app kiosk info to the user manager

The user manager already has access to the cloud print interface, which
contains information about whether cookie based printers are disabled
due to app kiosk mode. Instead of passing the app kiosk information
from the sidebar to the destination settings, allow the user manager to
access the desired information through a getter added to the interface.

Change-Id: I7669d99fcfa2c8228de6dff42fff0e11ce1fb0c7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904946
Commit-Queue: Daniel Hosseinian <dhoss@chromium.org>
Reviewed-by: Rebekah Potter <rbpotter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#714752}

chrome/browser/resources/print_preview/cloud_print_interface.js

@@ -71,6 +71,9 @@ import {Invitation} from './data/invitation.js';
 
   /** @interface */
   export class CloudPrintInterface {
+    /** @return {boolean} Whether cookie destinations are disabled. */
+    areCookieDestinationsDisabled() {}
+
     /**
      * @return {boolean} Whether a search for cloud destinations is in progress.
      */

chrome/browser/resources/print_preview/cloud_print_interface_js.js

@@ -38,8 +38,8 @@ import {parseCloudDestination, parseInvitation} from './data/cloud_parsers.js';
       this.nativeLayer_ = nativeLayer;
 
       /**
-       * Whether Print Preview is in App Kiosk mode, basically, use only
-       * printers available for the device.
+       * Whether Print Preview is in App Kiosk mode; use only printers available
+       * for the device and disable cookie destinations.
        * @private {boolean}
        */
       this.isInAppKioskMode_ = isInAppKioskMode;
@@ -85,6 +85,11 @@ import {parseCloudDestination, parseInvitation} from './data/cloud_parsers.js';
       this.eventTarget_ = new EventTarget();
     }
 
+    /** @override */
+    areCookieDestinationsDisabled() {
+      return this.isInAppKioskMode_;
+    }
+
     /** @override */
     getEventTarget() {
       return this.eventTarget_;

chrome/browser/resources/print_preview/cloud_print_interface_native.js

@@ -8,6 +8,9 @@ import {CloudPrintInterface} from './cloud_print_interface.js';
   export class CloudPrintInterfaceNative {
     constructor() {}
 
+    /** @override */
+    areCookieDestinationsDisabled() {}
+
     /** @override */
     isCloudDestinationSearchInProgress() {}
 

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

@@ -30,8 +30,6 @@ Polymer({
       notify: true,
     },
 
-    appKioskMode: Boolean,
-
     cloudPrintDisabled: {
       type: Boolean,
       value: true,
@@ -134,7 +132,8 @@ Polymer({
    * @private
    */
   checkCloudPrintStatus_: function(event) {
-    if (event.detail.status != 403 || this.appKioskMode) {
+    if (event.detail.status != 403 ||
+        this.cloudPrintInterface.areCookieDestinationsDisabled()) {
       return;
     }
 

chrome/browser/resources/print_preview/ui/destination_settings.html

@@ -34,7 +34,6 @@
       }
     </style>
     <print-preview-user-manager id="userManager" active-user="{{activeUser_}}"
-        app-kiosk-mode="[[appKioskMode]]"
         cloud-print-disabled="{{cloudPrintDisabled_}}"
         cloud-print-interface="[[cloudPrintInterface]]"
         users="{{users_}}" destination-store="[[destinationStore_]]"

chrome/browser/resources/print_preview/ui/destination_settings.js

@@ -49,8 +49,6 @@ import '../strings.m.js';
     ],
 
     properties: {
-      appKioskMode: Boolean,
-
       /** @type {CloudPrintInterface} */
       cloudPrintInterface: {
         type: Object,
@@ -265,7 +263,7 @@ import '../strings.m.js';
     init: function(
         defaultPrinter, pdfPrinterDisabled,
         serializedDefaultDestinationRulesStr, userAccounts, syncAvailable) {
-      this.pdfPrinterDisabled_ = this.appKioskMode || pdfPrinterDisabled;
+      this.pdfPrinterDisabled_ = pdfPrinterDisabled;
       this.$.userManager.initUserAccounts(userAccounts, syncAvailable);
       this.destinationStore_.init(
           this.pdfPrinterDisabled_, defaultPrinter,

chrome/browser/resources/print_preview/ui/sidebar.js

@@ -145,8 +145,9 @@ Polymer({
       appKioskMode, defaultPrinter, serializedDestinationSelectionRulesStr,
       userAccounts, syncAvailable, pdfPrinterDisabled) {
     this.isInAppKioskMode_ = appKioskMode;
+    const saveAsPdfDisabled = this.isInAppKioskMode_ || pdfPrinterDisabled;
     this.$.destinationSettings.init(
-        defaultPrinter, pdfPrinterDisabled,
+        defaultPrinter, saveAsPdfDisabled,
         serializedDestinationSelectionRulesStr, userAccounts, syncAvailable);
   },