diff options
| author | vitalybuka@chromium.org <vitalybuka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-07 23:48:37 +0000 |
|---|---|---|
| committer | vitalybuka@chromium.org <vitalybuka@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-07 23:48:37 +0000 |
| commit | 702484bde5a044d224230f08ea12aeeb7cd069d6 (patch) | |
| tree | 351e4cd5286f8eb82454289e027c696903fb03f3 /printing | |
| parent | 35f156649669568f85f992745a4528e829b7f921 (diff) | |
| download | chromium_src-702484bde5a044d224230f08ea12aeeb7cd069d6.zip chromium_src-702484bde5a044d224230f08ea12aeeb7cd069d6.tar.gz chromium_src-702484bde5a044d224230f08ea12aeeb7cd069d6.tar.bz2 | |
Revert 275646 "Generalize printer color model handling, get rid ..."
Speculative revert.
Something breaks "Linux ASan LSan Tests", TaskManagerNewPrintPreview.
> Generalize printer color model handling, get rid of CUPS specific case. Enable supported paper szes reporting for all platforms.
>
> BUG=239879
>
> Review URL: https://codereview.chromium.org/324523002
TBR=alekseys@chromium.org
Review URL: https://codereview.chromium.org/319373004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@275711 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'printing')
| -rw-r--r-- | printing/backend/print_backend.cc | 5 | ||||
| -rw-r--r-- | printing/backend/print_backend.h | 10 | ||||
| -rw-r--r-- | printing/backend/print_backend_win.cc | 2 |
3 files changed, 14 insertions, 3 deletions
diff --git a/printing/backend/print_backend.cc b/printing/backend/print_backend.cc index fefa789..d071201 100644 --- a/printing/backend/print_backend.cc +++ b/printing/backend/print_backend.cc @@ -19,9 +19,12 @@ PrinterSemanticCapsAndDefaults::PrinterSemanticCapsAndDefaults() duplex_capable(false), duplex_default(UNKNOWN_DUPLEX_MODE), color_changeable(false), - color_default(false), + color_default(false) +#if defined (OS_POSIX) + , color_model(UNKNOWN_COLOR_MODEL), bw_model(UNKNOWN_COLOR_MODEL) +#endif {} PrinterSemanticCapsAndDefaults::~PrinterSemanticCapsAndDefaults() {} diff --git a/printing/backend/print_backend.h b/printing/backend/print_backend.h index 3b307f1..30e4512 100644 --- a/printing/backend/print_backend.h +++ b/printing/backend/print_backend.h @@ -48,18 +48,28 @@ struct PRINTING_EXPORT PrinterSemanticCapsAndDefaults { bool color_changeable; bool color_default; + + // These are CUPS specific data, which soon be removed altogether. They are + // not defined under USE_CUPS to do not pull CUPS dependency into common code. +#if defined(OS_POSIX) + // TODO(alekseys): Resolve color model within printing context, do not expose + // it outside of the context. ColorModel color_model; ColorModel bw_model; +#endif +#if defined(OS_WIN) struct Paper { std::string name; gfx::Size size_um; }; + std::vector<Paper> papers; Paper default_paper; std::vector<gfx::Size> dpis; gfx::Size default_dpi; +#endif }; struct PRINTING_EXPORT PrinterCapsAndDefaults { diff --git a/printing/backend/print_backend_win.cc b/printing/backend/print_backend_win.cc index 87c69ee..75c562a 100644 --- a/printing/backend/print_backend_win.cc +++ b/printing/backend/print_backend_win.cc @@ -267,8 +267,6 @@ bool PrintBackendWin::GetPrinterSemanticCapsAndDefaults( // http://msdn.microsoft.com/en-us/library/windows/desktop/dd183552(v=vs.85).aspx caps.color_changeable = (DeviceCapabilities(name, port, DC_COLORDEVICE, NULL, NULL) == 1); - caps.color_model = printing::COLOR; - caps.bw_model = printing::GRAY; caps.duplex_capable = (DeviceCapabilities(name, port, DC_DUPLEX, NULL, NULL) == 1); |