Print preview page selection should not require a rerendering of draft pages.

chrome/test/data/webui/print_preview.js

 // Copyright (c) 2011 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.
 
 /**
  * Test fixture for print preview WebUI testing.
  * @extends {testing.Test}
  * @constructor
  */
 function PrintPreviewWebUITest() {}
@@ skipping 21 matching lines @@
     /**
      * Create a handler class with empty methods to allow mocking to register
      * expectations and for registration of handlers with chrome.send.
      * @constructor
      */
     function MockPrintPreviewHandler() {}
 
     MockPrintPreviewHandler.prototype = {
       getDefaultPrinter: function() {},
       getPrinters: function() {},
-      getPreview: function(settings) {},
+      getPreview: function(settings, draft_page_count, modifiable) {},
       print: function(settings) {},
       getPrinterCapabilities: function(printerName) {},
       showSystemDialog: function() {},
       morePrinters: function() {},
       signIn: function() {},
       addCloudPrinter: function() {},
       manageCloudPrinters: function() {},
       manageLocalPrinters: function() {},
       closePrintPreviewTab: function() {},
       hidePreview: function() {},
@@ skipping 12 matching lines @@
     mockHandler.stubs().getPrinterCapabilities(NOT_NULL).
         will(callFunction(function() {
           updateWithPrinterCapabilities({
             disableColorOption: true,
             setColorAsDefault: true,
             disableCopiesOption: true,
           });
         }));
     var savedArgs = new SaveMockArguments();
     mockHandler.stubs().getPreview(savedArgs.match(NOT_NULL)).
-        will(callFunctionWithSavedArgs(savedArgs, function(options) {
-          updatePrintPreview('title', true, 1, JSON.parse(options).requestID);
+        will(callFunctionWithSavedArgs(savedArgs, function(args) {
+          updatePrintPreview(1, JSON.parse(args[0]).requestID);
         }));
 
     mockHandler.stubs().getPrinters().
         will(callFunction(function() {
           setUseCloudPrint(false, '');
           setPrinters([{
               printerName: 'FooName',
               deviceName: 'FooDevice',
             }, {
               printerName: 'BarName',
@@ skipping 49 matching lines @@
   typedefCppFixture: null,
 
   /**
    * Create the PDF plugin or reload the existing one. This function replaces
    * createPDFPlugin defined in
    * chrome/browser/resources/print_preview/print_preview.js when there is no
    * official pdf plugin so that the WebUI logic can be tested. It creates and
    * attaches an HTMLDivElement to the |mainview| element with attributes and
    * empty methods, which are used by testing and that would be provided by the
    * HTMLEmbedElement when the PDF plugin exists.
-   * @param {string} previewUid Preview unique identifier.
+   * @param {number} srcDataIndex Preview data source index.
    */
-  createPDFPlugin: function(previewUid) {
+  createPDFPlugin: function(srcDataIndex) {
     var pdfViewer = $('pdf-viewer');
     if (pdfViewer)
       return;
 
+    var previewUid = 1;
     pdfViewer = document.createElement('div');
     pdfViewer.setAttribute('id', 'pdf-viewer');
     pdfViewer.setAttribute('type',
                            'application/x-google-chrome-print-preview-pdf');
     pdfViewer.setAttribute(
-        'src', 'chrome://print/' + previewUid + '/print.pdf');
+        'src',
+        'chrome://print/' + previewUid + '/' + srcDataIndex + '/print.pdf');
     pdfViewer.setAttribute('aria-live', 'polite');
     pdfViewer.setAttribute('aria-atomic', 'true');
     function fakeFunction() {}
     pdfViewer.onload = fakeFunction;
     pdfViewer.goToPage = fakeFunction;
     pdfViewer.removePrintButton = fakeFunction;
     pdfViewer.fitToHeight = fakeFunction;
     pdfViewer.grayscale = fakeFunction;
     $('mainview').appendChild(pdfViewer);
     onPDFLoad();
@@ skipping 165 matching lines @@
 
   updateControlsWithSelectedPrinterCapabilities();
   expectFalse(colorSettings.colorRadioButton.checked);
   expectTrue(colorSettings.bwRadioButton.checked);
 });
 
 // Test that changing the selected printer updates the preview.
 TEST_F('PrintPreviewWebUITest', 'TestPrinterChangeUpdatesPreview', function() {
   var savedArgs = new SaveMockArguments();
   this.mockHandler.expects(once()).getPreview(savedArgs.match(ANYTHING)).
-      will(callFunctionWithSavedArgs(savedArgs, function(options) {
-        updatePrintPreview('title', true, 2,
-                           JSON.parse(options).requestID);
+      will(callFunctionWithSavedArgs(savedArgs, function(args) {
+        updatePrintPreview(2, JSON.parse(args[0]).requestID);
       }));
 
   this.mockGlobals.expects(once()).updateWithPrinterCapabilities(
       savedArgs.match(ANYTHING)).
           will(callGlobalWithSavedArgs(
               savedArgs, 'updateWithPrinterCapabilities'));
 
   var printerList = $('printer-list');
   assertNotEquals(null, printerList, 'printerList');
   assertGE(printerList.options.length, printerListMinLength);
@@ skipping 34 matching lines @@
 
 // Test that error message is displayed when plugin doesn't exist.
 TEST_F('PrintPreviewNoPDFWebUITest', 'TestErrorMessage', function() {
   var errorButton = $('error-button');
   assertNotEquals(null, errorButton);
   expectFalse(errorButton.disabled);
   var errorText = $('error-text');
   assertNotEquals(null, errorText);
   expectFalse(errorText.classList.contains('hidden'));
 });