Print Preview: Update test to check parameters sent to print()

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 {TestBrowserProxy}
   */
@@ skipping 80 matching lines @@
       return Promise.resolve(true);
     },
 
     /** @override */
     getPrinterCapabilities: function(printerId) {
       this.methodCalled('getPrinterCapabilities', printerId);
       return this.localDestinationCapabilities_.get(printerId);
     },
 
     /** @override */
-    print: function() {
-      this.methodCalled('print');
+    print: function(
+        destination, printTicketStore, cloudPrintInterface, documentInfo) {
+      var args = {"destination": destination,

[dpapad, 2017/06/27 18:49:00]

Nit1: Prefer single quotes instead of double quotes in JS.
Nit2: The quotes are not necessary at all here.
Nit3: Perhaps inline as followls

this.methodCalled('print', {
  destination: destination,
  printTicketStore: printTicketStore,
  ...
});

[rbpotter, 2017/06/28 01:14:57]

Done.

+                  "printTicketStore": printTicketStore,
+                  "cloudPrintInterface": cloudPrintInterface,
+                  "documentInfo": documentInfo };
+      this.methodCalled('print', args);
       return Promise.resolve();
     },
 
     /** @override */
     setupPrinter: function(printerId) {
       this.methodCalled('setupPrinter', printerId);
       return this.shouldRejectPrinterSetup_ ?
           Promise.reject(this.setupPrinterResponse_) :
           Promise.resolve(this.setupPrinterResponse_);
     },
@@ skipping 54 matching lines @@
     setSetupPrinterResponse: function(reject, response) {
       this.shouldRejectPrinterSetup_ = reject;
       this.setupPrinterResponse_ = response;
     },
   };
 
   return {
     NativeLayerStub: NativeLayerStub,
   };
 });