diff options
author | ckocagil@chromium.org <ckocagil@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-08 07:56:36 +0000 |
---|---|---|
committer | ckocagil@chromium.org <ckocagil@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-08 07:56:36 +0000 |
commit | 1e4771760a9244d2cef1c544299b976b9f940d12 (patch) | |
tree | f65bb82fa126279d7a6bb47d24205185e5dbfc62 /chrome/browser/resources/print_preview/native_layer.js | |
parent | 64bb94fdffc91abd7ab5c2ba8a1054ff445f1467 (diff) | |
download | chromium_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.js | 23 |
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? |