PrintPreview: Fixed RICOH MP C3501 color print job issues.

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 c0547e90ea43ab39f624f49f7b67f88841a6cbfc..bf5cedb998243fae66ecab539ff6ddc42a5c2f54 100644
--- a/chrome/browser/resources/print_preview/color_settings.js
+++ b/chrome/browser/resources/print_preview/color_settings.js
@@ -14,6 +14,10 @@ cr.define('print_preview', function() {
     this.colorOption_ = $('color-option');
     this.colorRadioButton_ = $('color');
     this.bwRadioButton_ = $('bw');
+    this.GRAY = 1;
+    this.CMYK = 3; // cmyk - Cyan, magenta, yellow, black

[vandebo (ex-Chrome), 2011/09/04 17:24:39]

nit: two spaces before an inline comment.

[kmadhusu, 2011/09/05 17:37:20]

Done.

+    this.COLOR = 2;

[vandebo (ex-Chrome), 2011/09/04 17:24:39]

nit: put these in numerical order - gray, color, cmyk

[kmadhusu, 2011/09/05 17:37:20]

Done.

+    this.printerColorModelForColor_ = this.COLOR;
   }
 
   cr.addSingletonGetter(ColorSettings);
@@ -36,11 +40,13 @@ cr.define('print_preview', function() {
     },
 
     /**
-     * Checks whether |this.colorRadioButton_| is checked.
-     * @return {boolean} true if |this.colorRadioButton_| is checked.
+     * Returns the color mode for print preview.
+     * @return {Number} Returns the printer color space
      */
-    isColor: function() {
-      return this.colorRadioButton_.checked;
+    get colorMode() {
+      if (this.bwRadioButton_.checked)
+        return this.GRAY;
+      return this.printerColorModelForColor_;

[dpapad, 2011/09/04 04:55:16]

Nit (optional): return this.bwRadioButton_.checked ? this.GRAY :
this.printerColorModelForColor_;

[vandebo (ex-Chrome), 2011/09/04 17:24:39]

There's no particular reason to use a tristate here

     },
 
     /**
@@ -71,6 +77,8 @@ cr.define('print_preview', function() {
       this.colorOption_.setAttribute('aria-hidden', disableColorOption);
 
       var setColorAsDefault = e.printerCapabilities.setColorAsDefault;
+      this.printerColorModelForColor_ =
+          e.printerCapabilities.printerColorModelForColor;
       this.colorRadioButton_.checked = setColorAsDefault;
       this.bwRadioButton_.checked = !setColorAsDefault;
       setColor(this.colorRadioButton_.checked);