PrintPreview: Fixed RICOH MP C3501 color print job issues.

chrome/browser/resources/print_preview/print_preview.js

Index: chrome/browser/resources/print_preview/print_preview.js
diff --git a/chrome/browser/resources/print_preview/print_preview.js b/chrome/browser/resources/print_preview/print_preview.js
index 51838a46e9a13ec51eb8a9227f3ed93cea4d018b..e2c1e777b7ffdc8747b837dced2f808f98f06019 100644
--- a/chrome/browser/resources/print_preview/print_preview.js
+++ b/chrome/browser/resources/print_preview/print_preview.js
@@ -237,6 +237,7 @@ function updateControlsWithSelectedPrinterCapabilities() {
         'disableColorOption': true,
         'setColorAsDefault': true,
         'setDuplexAsDefault': false,
+        'printerColorSpace': colorSettings.color_,

[dpapad, 2011/09/02 22:15:06]

In general we do not allow accessing member vars outside the class without a
getter. See my other comment about how to name constants.

[kmadhusu, 2011/09/04 01:52:51]

Done.

         'disableCopiesOption': true});
   } else {
     // This message will call back to 'updateWithPrinterCapabilities'
@@ -344,7 +345,7 @@ function getSettings() {
        'copies': copiesSettings.numberOfCopies,
        'collate': copiesSettings.isCollated(),
        'landscape': layoutSettings.isLandscape(),
-       'color': colorSettings.isColor(),
+       'color': colorSettings.getColorMode(),

[dpapad, 2011/09/02 22:15:06]

Replace with "colorSettings.colorMode" after you use the proper get syntax.

[kmadhusu, 2011/09/04 01:52:51]

Done.

        'printToPDF': printToPDF,
        'isFirstRequest' : false,
        'headerFooterEnabled': headerFooterSettings.hasHeaderFooter(),
@@ -1023,7 +1024,7 @@ function createPDFPlugin(srcDataIndex) {
     pdfViewer.goToPage('0');
     pdfViewer.resetPrintPreviewUrl(srcURL);
     pdfViewer.reload();
-    pdfViewer.grayscale(!colorSettings.isColor());
+    pdfViewer.grayscale(colorSettings.getColorMode() == colorSettings.gray_);
     return;
   }