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

chrome/browser/resources/print_preview/print_preview.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.
 
 // TODO(rltoscano): Move data/* into print_preview.data namespace
 
 var localStrings = new LocalStrings(templateData);
 
 <include src="component.js"/>
 
@@ skipping 267 matching lines @@
           print_preview.NativeLayer.EventType.SETTINGS_INVALID,
           this.onSettingsInvalid_.bind(this));
       this.tracker.add(
           this.nativeLayer_,
           print_preview.NativeLayer.EventType.DISABLE_SCALING,
           this.onDisableScaling_.bind(this));
       this.tracker.add(
           this.nativeLayer_,
           print_preview.NativeLayer.EventType.PRIVET_PRINT_FAILED,
           this.onPrivetPrintFailed_.bind(this));
-
+      this.tracker.add(
+          this.nativeLayer_,
+          print_preview.NativeLayer.EventType.MANIPULATE_SETTINGS,
+          this.onManipulateSettings_.bind(this));
 
       this.tracker.add(
           $('system-dialog-link'),
           'click',
           this.openSystemPrintDialog_.bind(this));
       this.tracker.add(
           $('cloud-print-dialog-link'),
           'click',
           this.onCloudPrintDialogLinkClick_.bind(this));
       this.tracker.add(
@@ skipping 385 matching lines @@
       this.isPreviewGenerationInProgress_ = true;
     },
 
     /**
      * Called when the preview area's preview generation is complete.
      * @private
      */
     onPreviewGenerationDone_: function() {
       this.isPreviewGenerationInProgress_ = false;
       this.printHeader_.isPrintButtonEnabled = true;
+      if (global['onManipulateSettings']) {

[ivandavid, 2014/06/17 22:59:16]

This makes it so that the chrome.send() is only used when doing this particular
test.

+        chrome.send('UILoadedForTest');

[Lei Zhang, 2014/06/17 23:22:20]

Although this is just 1 line, consider putting it in a function with a
meaningful name and calling it everywhere, so you avoid typos in the message
name.

[ivandavid, 2014/06/18 00:25:59]

Done. Made a function in native_layer.js since that is where all the
chrome.send() functions seem to be. Its called |doneManipulatingSettings()|.

+      }
       this.printIfReady_();
     },
 
     /**
      * Called when the preview area's preview failed to load.
      * @private
      */
     onPreviewGenerationFail_: function() {
       this.isPreviewGenerationInProgress_ = false;
       this.printHeader_.isPrintButtonEnabled = false;
@@ skipping 178 matching lines @@
      * @param {Event} event Event object representing the failure.
      * @private
      */
     onPrivetPrintFailed_: function(event) {
       console.error('Privet printing failed with error code ' +
                     event.httpError);
       this.printHeader_.setErrorMessage(
         localStrings.getString('couldNotPrint'));
     },
 
+    onManipulateSettings_: function(event) {
+      if (event.messageName == 'SAVE_AS_PDF') {
+        this.destinationStore_.selectDefaultDestination_();
+      }
+      else if (event.messageName == 'LAYOUT_SETTINGS') {
+        if (event.layoutSettings == 'LANDSCAPE') {
+          var landscape = document.getElementsByClassName(
+            'layout-settings-landscape-radio')[0];
+          if (landscape) {

[Lei Zhang, 2014/06/17 23:22:20]

So if your source page is a PDF and not HTML, then I believe there's no layout
control. In that case, this will eval to false and ultimately no UILoadedForTest
message will get sent. That would cause the state machine in the test to not
advance.

+            landscape.click();
+          }
+        } else if (event.layoutSettings == 'PORTRAIT') {
+          //chrome.send('UILoadedForTest');

[Lei Zhang, 2014/06/17 23:22:20]

remove

+          var portrait = document.getElementsByClassName(
+            'layout-settings-portrait-radio')[0];
+          if (portrait) {
+            portrait.click();
+          }
+        }
+      }
+      else if (event.messageName == 'PAGE_NUMBERS') {
+        if (event.pageNumbers != '') {
+          var pageSettingsCustomInput = document.getElementsByClassName(
+              'page-settings-custom-input')[0];
+          if (pageSettingsCustomInput) {
+            pageSettingsCustomInput.value = event.pageSettings;
+          }
+        } else {
+          chrome.send('UILoadedForTest');
+        }
+      }
+      else if (event.messageName == 'HEADERS_AND_FOOTERS') {
+        if (event.headersAndFooters == false) {

[Lei Zhang, 2014/06/17 23:22:20]

There's 2x2 matrix of possibilities, where the axis are
|event.headersAndFooters| and |headerFooterCheckbox|'s current value. You should
handle all the cases.

Same for background colors below.

+          var headerFooterCheckbox = document.getElementsByClassName(
+              'header-footer-checkbox');
+          if (headerFooterCheckbox) {
+            this.otherOptionsSettings_.headerFooterCheckbox_.click();
+          }
+        } else {
+          chrome.send('UILoadedForTest');
+        }
+      }
+      else if (event.messageName == 'BACKGROUND_COLORS_AND_IMAGES') {
+        if (event.backgroundColorsAndImages == true) {
+          var backgroundCheckbox = document.getElementsByClassName(
+              'css-background-checkbox')[0];
+          if (backgroundCheckbox) {
+            this.otherOptionsSettings_.cssBackgroundCheckbox_.click();
+          }
+        } else {
+          chrome.send('UILoadedForTest');
+        }
+      }
+      else if (event.messageName == 'MARGINS') {
+        var marginSettingsSelect =
+            document.getElementsByClassName('margin-settings-select')[0];
+        var index = parseInt(event.margins, 10);
+        if (marginSettingsSelect && index != 0) {
+           marginSettingsSelect.selectedIndex = index;

[Lei Zhang, 2014/06/17 23:22:20]

Is this sufficient to cause the PDF to re-render? If so, you probably shouldn't
need to do a chrome.send() on the next line?

[ivandavid, 2014/06/18 00:25:59]

It wasn't. I 'fired' on event which now causes the PDF to re-render.

+           chrome.send('UILoadedForTest');

[Lei Zhang, 2014/06/17 23:22:20]

nit: If you compare this to line 959, one of the two blocks is badly indented.

+        } else {
+          chrome.send('UILoadedForTest');
+        }
+      }
+    },
+
     /**
      * Called when the open-cloud-print-dialog link is clicked. Opens the Google
      * Cloud Print web dialog.
      * @private
      */
     onCloudPrintDialogLinkClick_: function() {
       assert(this.uiState_ == PrintPreview.UiState_.READY,
              'Opening Google Cloud Print dialog when not in ready state: ' +
                  this.uiState_);
       setIsVisible($('cloud-print-dialog-throbber'), true);
@@ skipping 132 matching lines @@
 <include src="search/recent_destination_list.js"/>
 <include src="search/destination_list_item.js"/>
 <include src="search/destination_search.js"/>
 <include src="search/search_box.js"/>
 <include src="search/fedex_tos.js"/>
 
 window.addEventListener('DOMContentLoaded', function() {
   printPreview = new print_preview.PrintPreview();
   printPreview.initialize();
 });