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..3b1bd2273ded75b2182cfd37a8812dab9f5699b4 100644
--- a/chrome/browser/resources/print_preview/print_preview_utils.js
+++ b/chrome/browser/resources/print_preview/print_preview_utils.js
@@ -189,3 +189,15 @@ function pageSetToPageRanges(pageSet) {
 function getPageSrcURL(id, pageNumber) {
   return 'chrome://print/' + id + '/' + pageNumber + '/print.pdf';
 }
+
+
+/**
+ * Returns a random integer within the specified range, |from| and |to| are
+ * included.
+ * @param {number} from Start of the range.
+ * @param {number} to  End of the range.
+ * @return {number} The random integer.
+ */
+function getRandomIntegerWithinRange(from, to) {

[kmadhusu, 2011/08/09 22:30:40]

This function can return negative value. Please validate from & to and make sure
Math.floor returns a positive integer. 

[dpapad, 2011/08/10 00:17:11]

This function does what it says in the documentation. It can return negative
numbers if |from| is negative. It is up to the caller to use it appropriately.
Why constraining it to return a positive integer?

I did fix it though so that the order of arguments does not matter
(getRandomIntegerWithinRange(0,10) and getRandomIntegerWithinRange(10,0) both
work as expected now).
I am also planning to a new test for this function.

+  return Math.floor(Math.random() * (to - from + 1) + from);
+}