Print Preview: Fixing the behavior of increment/decrement buttons, destinations drop-down list.

chrome/browser/resources/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.
 
 var localStrings = new LocalStrings();
 
 // The total page count of the previewed document regardless of which pages the
 // user has selected.
 var totalPageCount = -1;
 
@@ skipping 567 matching lines @@
   resetPageRangeFieldTimer();
   updatePrintButtonState();
   updatePrintSummary();
 }
 
 /**
  * Updates the state of the increment/decrement buttons based on the current
  * 'copies' value.
  */
 function updateCopiesButtonsState() {
+  var copiesField = $('copies');
   if (!isNumberOfCopiesValid()) {
-    $('copies').classList.add('invalid');
-    $('increment').disabled = true;
-    $('decrement').disabled = true;
+    copiesField.classList.add('invalid');
+    $('increment').disabled = false;
+    $('decrement').disabled = false;
     showInvalidHint($('copies-hint'));
   }
   else {
-    $('copies').classList.remove('invalid');
-    $('increment').disabled = false;
-    $('decrement').disabled = false;
+    copiesField.classList.remove('invalid');
+    $('increment').disabled = (getCopies() == copiesField.max) ? true : false;
+    $('decrement').disabled = (getCopies() == copiesField.min) ? true : false;
     hideInvalidHint($('copies-hint'));
   }
 }
 
 /**
  * Updates the print summary based on the currently selected user options.
  *
  */
 function updatePrintSummary() {
   var copies = getCopies();
@@ skipping 277 matching lines @@
   for (var i = 0; i < array1.length; i++)
     if(array1[i] != array2[i])
       return false;
   return true;
 }
 
 /**
  * Executed when the 'increment' or 'decrement' button is clicked.
  */
 function onCopiesButtonsClicked(sign) {
-  if($('copies').value == 1 && (sign == -1))
-    return;
-  $('copies').value = getCopies() + sign * 1;
+  var copiesField = $('copies');
+  if (!isNumberOfCopiesValid())
+    copiesField.value = 1;
+  else {
+    var newValue = getCopies() + sign * 1;
+    if (newValue < copiesField.min || newValue > copiesField.max)
+      return;
+    copiesField.value = newValue;
+  }
   copiesFieldChanged();
 }