Page range comparisons should use document size.

chrome/browser/resources/print_preview/data/print_ticket_store.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';
 
   // TODO(rltoscano): Maybe clear print ticket when destination changes. Or
   // better yet, carry over any print ticket state that is possible. I.e. if
   // destination changes, the new destination might not support duplex anymore,
@@ skipping 33 matching lines @@
     // will be valid even if no preview can be generated.
     var initialPageSize = new print_preview.Size(612, 792); // 8.5"x11"
 
     /**
      * Information about the document to print.
      * @type {!print_preview.DocumentInfo}
      * @private
      */
     this.documentInfo_ = new print_preview.DocumentInfo();
     this.documentInfo_.isModifiable = true;
-    this.documentInfo_.pageCount = 1;
+    this.documentInfo_.pageCount = 0;
     this.documentInfo_.pageSize = initialPageSize;
     this.documentInfo_.printableArea = new print_preview.PrintableArea(
         new print_preview.Coordinate2d(0, 0), initialPageSize);
 
     /**
      * Printing capabilities of Chromium and the currently selected destination.
      * @type {!print_preview.CapabilitiesHolder}
      * @private
      */
     this.capabilitiesHolder_ = new print_preview.CapabilitiesHolder();
@@ skipping 521 matching lines @@
     isPageRangeValid: function() {
       return this.pageRange_.isValid();
     },
 
     /** @return {string} String representation of the page range. */
     getPageRangeStr: function() {
       return this.pageRange_.getValue();
     },
 
     /**
+     * @return {!Array.<object.<{from: number, to: number}>>} Page ranges
+     *     represented by of the page range string.
+     */
+    getPageRanges: function() {
+      return this.pageRange_.getPageRanges();
+    },
+
+    /**
+     * @return {!Array.<object.<{from: number, to: number}>>} Page ranges
+     *     represented by of the page range string and constraied by ducument
+     *     page count.
+     */
+    getDocumentPageRanges: function() {
+      return this.pageRange_.getDocumentPageRanges();
+    },
+
+    /**
      * @return {!print_preview.PageNumberSet} Page number set specified by the
      *     string representation of the page range string.
      */
     getPageNumberSet: function() {
       return this.pageRange_.getPageNumberSet();
     },
 
     /**
      * Updates the page range string. Dispatches a TICKET_CHANGE if the string
      * changed.
@@ skipping 138 matching lines @@
             this, PrintTicketStore.EventType.CAPABILITIES_CHANGE);
       }
     }
   };
 
   // Export
   return {
     PrintTicketStore: PrintTicketStore
   };
 });