summaryrefslogtreecommitdiffstats
path: root/chrome/browser/resources/print_preview/native_layer.js
diff options
context:
space:
mode:
authorckocagil@chromium.org <ckocagil@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-06-08 07:56:36 +0000
committerckocagil@chromium.org <ckocagil@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-06-08 07:56:36 +0000
commit1e4771760a9244d2cef1c544299b976b9f940d12 (patch)
treef65bb82fa126279d7a6bb47d24205185e5dbfc62 /chrome/browser/resources/print_preview/native_layer.js
parent64bb94fdffc91abd7ab5c2ba8a1054ff445f1467 (diff)
downloadchromium_src-1e4771760a9244d2cef1c544299b976b9f940d12.zip
chromium_src-1e4771760a9244d2cef1c544299b976b9f940d12.tar.gz
chromium_src-1e4771760a9244d2cef1c544299b976b9f940d12.tar.bz2
Revert of Generalize printer color model handling, get rid of CUPS specific case. Enable supported paper szes… (https://codereview.chromium.org/324523002/)
Reason for revert: Breaks Linux ASAN http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Tests%20%281%29/builds/2865 Original issue's description: > Generalize printer color model handling, get rid of CUPS specific case. Enable supported paper szes reporting for all platforms. > > BUG=239879 > NOTRY=true > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275646 > > Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=275716 TBR=vitalybuka@chromium.org,jschuh@chromium.org,alekseys@chromium.org NOTREECHECKS=true NOTRY=true BUG=239879 Review URL: https://codereview.chromium.org/321703002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275739 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/resources/print_preview/native_layer.js')
-rw-r--r--chrome/browser/resources/print_preview/native_layer.js23
1 files changed, 4 insertions, 19 deletions
diff --git a/chrome/browser/resources/print_preview/native_layer.js b/chrome/browser/resources/print_preview/native_layer.js
index 0111ccd..efdd55b 100644
--- a/chrome/browser/resources/print_preview/native_layer.js
+++ b/chrome/browser/resources/print_preview/native_layer.js
@@ -168,23 +168,6 @@ cr.define('print_preview', function() {
},
/**
- * @param {!print_preview.Destination} destination Destination to print to.
- * @param {!print_preview.ticket_items.Color} color Color ticket item.
- * @return {number} Native layer color model.
- * @private
- */
- getNativeColorModel_: function(destination, color) {
- // For non-local printers native color model is ignored anyway.
- var option = destination.isLocal ? color.getSelectedOption() : null;
- var nativeColorModel = parseInt(option ? option.vendor_id : null);
- if (isNaN(nativeColorModel)) {
- return color.getValue() ?
- NativeLayer.ColorMode_.COLOR : NativeLayer.ColorMode_.GRAY;
- }
- return nativeColorModel;
- },
-
- /**
* Requests that a preview be generated. The following events may be
* dispatched in response:
* - PAGE_COUNT_READY
@@ -207,7 +190,8 @@ cr.define('print_preview', function() {
'pageRange': printTicketStore.pageRange.getDocumentPageRanges(),
'mediaSize': printTicketStore.mediaSize.getValue(),
'landscape': printTicketStore.landscape.getValue(),
- 'color': this.getNativeColorModel_(destination, printTicketStore.color),
+ 'color': printTicketStore.color.getValue() ?
+ NativeLayer.ColorMode_.COLOR : NativeLayer.ColorMode_.GRAY,
'headerFooterEnabled': printTicketStore.headerFooter.getValue(),
'marginsType': printTicketStore.marginsType.getValue(),
'isFirstRequest': requestId == 0,
@@ -279,7 +263,8 @@ cr.define('print_preview', function() {
'pageRange': printTicketStore.pageRange.getDocumentPageRanges(),
'pageCount': printTicketStore.pageRange.getPageNumberSet().size,
'landscape': printTicketStore.landscape.getValue(),
- 'color': this.getNativeColorModel_(destination, printTicketStore.color),
+ 'color': printTicketStore.color.getValue() ?
+ NativeLayer.ColorMode_.COLOR : NativeLayer.ColorMode_.GRAY,
'headerFooterEnabled': printTicketStore.headerFooter.getValue(),
'marginsType': printTicketStore.marginsType.getValue(),
'generateDraftData': true, // TODO(rltoscano): What should this be?