Print Preview: Fixing behavior of event listeners.

chrome/browser/resources/print_preview/print_preview_utils.js

Index: chrome/browser/resources/print_preview/print_preview_utils.js
diff --git a/chrome/browser/resources/print_preview/print_preview_utils.js b/chrome/browser/resources/print_preview/print_preview_utils.js
index 68e52243603b894dbc4cc06aec757c8bae2137f9..d759ed19a977551ed675203e0ce55511fc756e0a 100644
--- a/chrome/browser/resources/print_preview/print_preview_utils.js
+++ b/chrome/browser/resources/print_preview/print_preview_utils.js
@@ -189,3 +189,17 @@ function pageSetToPageRanges(pageSet) {
 function getPageSrcURL(id, pageNumber) {
   return 'chrome://print/' + id + '/' + pageNumber + '/print.pdf';
 }
+
+
+/**
+ * Returns a random integer within the specified range, |endPointA| and
+ * |endPointB| are included.
+ * @param {number} endPointA One end of the desired range.
+ * @param {number} endPointB  The other end of the desired range.
+ * @return {number} The random integer.
+ */
+function getRandomIntegerWithinRange(endPointA, endPointB) {

[Lei Zhang, 2011/08/10 03:26:07]

I think most people call this function "randInt()".

[dpapad, 2011/08/10 15:47:10]

Renamed to randomInteger() instead of randInt(). According to the C++ styleguide
"Function names, variable names, and filenames should be descriptive; eschew
abbreviation". I think it is a good practice to apply in JS too.

+  from = Math.min(endPointA, endPointB);
+  to = Math.max(endPointA, endPointB);
+  return Math.floor(Math.random() * (to - from + 1) + from);
+}