Print Preview: Fixing behavior of event listeners.

chrome/browser/resources/print_preview/color_settings.js

Index: chrome/browser/resources/print_preview/color_settings.js
diff --git a/chrome/browser/resources/print_preview/color_settings.js b/chrome/browser/resources/print_preview/color_settings.js
index bb0198ceb4b8c3289fb04f5553c719df31f87065..c0547e90ea43ab39f624f49f7b67f88841a6cbfc 100644
--- a/chrome/browser/resources/print_preview/color_settings.js
+++ b/chrome/browser/resources/print_preview/color_settings.js
@@ -49,12 +49,10 @@ cr.define('print_preview', function() {
      */
     addEventListeners: function() {
       this.colorRadioButton_.onclick = function() {

[kmadhusu, 2011/08/08 22:20:45]

Can you make this this.colorRadioButton_.onclick = setColor(true);

[dpapad, 2011/08/08 23:41:43]

No this is syntactically wrong. setColor(true) does not return a function in
order to assign it to onclick.

-        if (!hasPendingPreviewRequest)
-          setColor(true);
+        setColor(true);
       };
       this.bwRadioButton_.onclick = function() {
-        if (!hasPendingPreviewRequest)
-          setColor(false);
+        setColor(false);
       };
       document.addEventListener('PDFLoaded', this.onPDFLoaded_.bind(this));
       document.addEventListener('printerCapabilitiesUpdated',