Print Preview: Change getPrinters to cr.sendWithPromise

chrome/test/data/webui/print_preview/native_layer_stub.js

 // Copyright 2017 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() {
   /**
   * Test version of the native layer.
   * @constructor
   * @extends {settings.TestBrowserProxy}
   */
   function NativeLayerStub() {
     settings.TestBrowserProxy.call(this, [
-        'getInitialSettings', 'setupPrinter' ]);
+        'getInitialSettings', 'getPrinters', 'setupPrinter' ]);
 
     /**
      * @private {!cr.EventTarget} The event target used for dispatching and
      *     receiving events.
      */
     this.eventTarget_ = new cr.EventTarget();
 
     /** @private {boolean} Whether the native layer has sent a print message. */
     this.printStarted_ = false;
 
     /**
      * @private {boolean} Whether the native layer has set the generate draft
      *      parameter when requesting an updated preview.
      */
     this.generateDraft_ = false;
 
     /**
      * @private {!print_preview.NativeInitialSettings} The initial settings
      *     to be used for the response to a |getInitialSettings| call.
      */
     this.initialSettings_ = null;
 
     /**
+     *
+     * @private {Array<print_preview.LocalDestinationInfo>} Local destination

[dpapad, 2017/06/01 19:01:53]

?Array<!

Or set initial value to an empty array, and annotate as "!Array<!..."

[rbpotter, 2017/06/01 22:31:12]

Done.

+     *     list to be used for the response to |getPrinters|.
+     */
+    this.localDestinationInfos_ = null;
+
+    /**
      * @private {!print_preview.PrinterSetupResponse} The response to be sent
      *     on a |setupPrinter| call.
      */
     this.setupPrinterResponse_ = null;
 
     /**
      * @private {boolean} Whether the printer setup request should be rejected.
      */
     this.shouldRejectPrinterSetup_ = false;
 
@@ skipping 13 matching lines @@
   NativeLayerStub.prototype = {
     __proto__: settings.TestBrowserProxy.prototype,
 
     /** @override */
     getInitialSettings: function() {
       this.methodCalled('getInitialSettings');
       return Promise.resolve(this.initialSettings_);
     },
 
     /** @override */
+    getPrinters: function() {
+      this.methodCalled('getPrinters');
+      return Promise.resolve(this.localDestinationInfos_);
+    },
+
+    /** @override */
     setupPrinter: function(printerId) {
       this.methodCalled('setupPrinter', printerId);
       return this.shouldRejectPrinterSetup_ ?
           Promise.reject(this.setupPrinterResponse_) :
           Promise.resolve(this.setupPrinterResponse_);
     },
 
     /** Stubs for |print_preview.NativeLayer| methods that call C++ handlers. */
     previewReadyForTest: function() {},
     startGetLocalDestinations: function() {},
@@ skipping 45 matching lines @@
 
     /**
      * @param {!print_preview.NativeInitialSettings} settings The settings
      *     to return as a response to |getInitialSettings|.
      */
     setInitialSettings: function(settings) {
       this.initialSettings_ = settings;
     },
 
     /**
+     * @param {!Array<print_preview.LocalDestinationInfo>} localDestinations
+     *     The local destinations to return as a response to |getPrinters|.
+     */
+    setLocalDestinations: function(localDestinations) {
+      this.localDestinationInfos_ = localDestinations;
+    },
+
+    /**
      * @param {boolean} reject Whether printSetup requests should be rejected.
      * @param {!print_preview.PrinterSetupResponse} The response to send when
      *     |setupPrinter| is called.
      */
     setSetupPrinterResponse: function(reject, response) {
       this.shouldRejectPrinterSetup_ = reject;
       this.setupPrinterResponse_ = response;
     },
   };
 
   return {
     NativeLayerStub: NativeLayerStub,
   };
 });