Added a test that currently is able to print to pdf.

chrome/browser/resources/print_preview/native_layer.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';
 
   /**
    * An interface to the native Chromium printing system layer.
    * @constructor
@@ skipping 28 matching lines @@
     global['onDidPreviewPage'] = this.onDidPreviewPage_.bind(this);
     global['updatePrintPreview'] = this.onUpdatePrintPreview_.bind(this);
     global['printScalingDisabledForSourcePDF'] =
         this.onPrintScalingDisabledForSourcePDF_.bind(this);
     global['onDidGetAccessToken'] = this.onDidGetAccessToken_.bind(this);
     global['autoCancelForTesting'] = this.autoCancelForTesting_.bind(this);
     global['onPrivetPrinterChanged'] = this.onPrivetPrinterChanged_.bind(this);
     global['onPrivetCapabilitiesSet'] =
         this.onPrivetCapabilitiesSet_.bind(this);
     global['onPrivetPrintFailed'] = this.onPrivetPrintFailed_.bind(this);
+    global['onEnableManipulateSettingsForTest'] =
+        this.onEnableManipulateSettingsForTest_.bind(this);

[Dan Beam, 2014/06/20 02:45:16]

all of these would be better as global.property instead of global['property']
(are these quoted so they're not renamed when compiling JavaScript?)

[ivandavid, 2014/06/20 21:31:44]

Done.

   };
 
   /**
    * Event types dispatched from the Chromium native layer.
    * @enum {string}
    * @const
    */
   NativeLayer.EventType = {
     ACCESS_TOKEN_READY: 'print_preview.NativeLayer.ACCESS_TOKEN_READY',
     CAPABILITIES_SET: 'print_preview.NativeLayer.CAPABILITIES_SET',
@@ skipping 11 matching lines @@
     PAGE_PREVIEW_READY: 'print_preview.NativeLayer.PAGE_PREVIEW_READY',
     PREVIEW_GENERATION_DONE:
         'print_preview.NativeLayer.PREVIEW_GENERATION_DONE',
     PREVIEW_GENERATION_FAIL:
         'print_preview.NativeLayer.PREVIEW_GENERATION_FAIL',
     PRINT_TO_CLOUD: 'print_preview.NativeLayer.PRINT_TO_CLOUD',
     SETTINGS_INVALID: 'print_preview.NativeLayer.SETTINGS_INVALID',
     PRIVET_PRINTER_CHANGED: 'print_preview.NativeLayer.PRIVET_PRINTER_CHANGED',
     PRIVET_CAPABILITIES_SET:
         'print_preview.NativeLayer.PRIVET_CAPABILITIES_SET',
-    PRIVET_PRINT_FAILED: 'print_preview.NativeLayer.PRIVET_PRINT_FAILED'
+    PRIVET_PRINT_FAILED: 'print_preview.NativeLayer.PRIVET_PRINT_FAILED',
+    MANIPULATE_SETTINGS_FOR_TEST:
+        'print_preview.NativeLayer.MANIPULATE_SETTINGS_FOR_TEST'

[Dan Beam, 2014/06/20 02:45:16]

alphabetize list

[ivandavid, 2014/06/20 21:31:44]

Done.

   };
 
   /**
    * Constant values matching printing::DuplexMode enum.
    * @enum {number}
    */
   NativeLayer.DuplexMode = {
     SIMPLEX: 0,
     LONG_EDGE: 1,
     UNKNOWN_DUPLEX_MODE: -1
@@ skipping 574 matching lines @@
     /**
      * @param {string} http_error The HTTP response code or -1 if not an HTTP
      *    error.
      * @private
      */
     onPrivetPrintFailed_: function(http_error) {
       var privetPrintFailedEvent =
             new Event(NativeLayer.EventType.PRIVET_PRINT_FAILED);
       privetPrintFailedEvent.httpError = http_error;
       this.dispatchEvent(privetPrintFailedEvent);
+    },
+
+    /**
+     * Function that allows for onManipulateSettings to be called
+     * from the native layer.

[Dan Beam, 2014/06/20 02:45:16]

@private

[ivandavid, 2014/06/20 21:31:44]

Done.

+     */
+    onEnableManipulateSettingsForTest_: function() {
+      global['onManipulateSettingsForTest'] =
+          this.onManipulateSettingsForTest_.bind(this);
+    },
+
+    /**
+     * Function that dispatches an event to print_preview.js to change
+     * a particular setting for print preview.
+     * @param {Dictionary} settings Dictionary containing the value to be

[Dan Beam, 2014/06/20 02:45:16]

s/Dictionary/Object/g

[Aleksey Shlyapnikov, 2014/06/20 17:27:22]

!Object, since you do not handle settings to be null.

[ivandavid, 2014/06/20 21:31:44]

Done.

+     *     changed and that value should be set to.

[Dan Beam, 2014/06/20 02:45:17]

@private

[ivandavid, 2014/06/20 21:31:44]

Done.

+     */
+    onManipulateSettingsForTest_: function(settings) {
+      var manipulateSettingsEvent =
+          new Event(NativeLayer.EventType.MANIPULATE_SETTINGS_FOR_TEST);
+      manipulateSettingsEvent.settings = settings;
+      this.dispatchEvent(manipulateSettingsEvent);
+    },
+
+    /**
+     * Function that sends a message to the test, letting it know that an
+     * option has been set to a particular value and that the change has
+     * finished modifying the preview area.
+     */
+    previewReady: function() {

[Aleksey Shlyapnikov, 2014/06/20 17:27:21]

previewReadyForTest

[ivandavid, 2014/06/20 21:31:44]

Done.

+      if (global['onManipulateSettingsForTest']) {

[Dan Beam, 2014/06/20 02:45:17]

no curlies for conditionals and conditional bodies that are 1 line, e.g.

  // all 1 line -> no curlies
  if (...)
    ...
  else if (...)
    ...
  else
    ...

or

  // conditional is 2 lines -> curlies
  if (... &&
      ...) {
    ...
  }

or

  // body is 2 lines -> curlies (enforced by language)
  if (...) {
    ...
    ...
  }

[ivandavid, 2014/06/20 21:31:44]

Done.

[ivandavid, 2014/06/20 21:31:44]

Done and applied this everywhere else where it happened.

+        chrome.send('UILoadedForTest');
+      }
+    },
+
+    /**
+     * Function that notifies the test that the option it tried to change
+     * had not been changed successfully.
+     */
+    previewFailed: function() {

[Aleksey Shlyapnikov, 2014/06/20 17:27:21]

previewFailedForTest

[ivandavid, 2014/06/20 21:31:44]

Done.

+      chrome.send('UIFailedLoadingForTest');
     }
   };
 
   /**
    * Initial settings retrieved from the native layer.
    * @param {boolean} isInKioskAutoPrintMode Whether the print preview should be
    *     in auto-print mode.
    * @param {string} thousandsDelimeter Character delimeter of thousands digits.
    * @param {string} decimalDelimeter Character delimeter of the decimal point.
    * @param {!print_preview.MeasurementSystem.UnitType} unitType Unit type of
@@ skipping 165 matching lines @@
       return this.serializedAppStateStr_;
     }
   };
 
   // Export
   return {
     NativeInitialSettings: NativeInitialSettings,
     NativeLayer: NativeLayer
   };
 });