Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(111)

Side by Side Diff: dm/DMSrcSink.h

Issue 2078623002: Support sRGB dsts in opt code (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Drop parens Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « dm/DM.cpp ('k') | dm/DMSrcSink.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2015 Google Inc. 2 * Copyright 2015 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #ifndef DMSrcSink_DEFINED 8 #ifndef DMSrcSink_DEFINED
9 #define DMSrcSink_DEFINED 9 #define DMSrcSink_DEFINED
10 10
(...skipping 197 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 class ColorCodecSrc : public Src { 208 class ColorCodecSrc : public Src {
209 public: 209 public:
210 enum Mode { 210 enum Mode {
211 // Mimic legacy behavior and apply no color correction. 211 // Mimic legacy behavior and apply no color correction.
212 kBaseline_Mode, 212 kBaseline_Mode,
213 213
214 // Color correct images into a specific dst color space. If you happen to have this 214 // Color correct images into a specific dst color space. If you happen to have this
215 // monitor, you're in luck! The unmarked outputs of this test should di splay 215 // monitor, you're in luck! The unmarked outputs of this test should di splay
216 // correctly on this monitor in the Chrome browser. If not, it's useful to know 216 // correctly on this monitor in the Chrome browser. If not, it's useful to know
217 // that this monitor has a profile that is fairly similar to Adobe RGB. 217 // that this monitor has a profile that is fairly similar to Adobe RGB.
218 // TODO (msarett): Should we add a new test with a new monitor and verif y that outputs
219 // look identical on two different dsts?
220 kDst_HPZR30w_Mode, 218 kDst_HPZR30w_Mode,
221 219
220 kDst_sRGB_Mode,
221
222 #if defined(SK_TEST_QCMS) 222 #if defined(SK_TEST_QCMS)
223 // Use QCMS for color correction. 223 // Use QCMS for color correction.
224 kQCMS_HPZR30w_Mode, 224 kQCMS_HPZR30w_Mode,
225 #endif 225 #endif
226 }; 226 };
227 227
228 ColorCodecSrc(Path, Mode); 228 ColorCodecSrc(Path, Mode);
229 229
230 Error draw(SkCanvas*) const override; 230 Error draw(SkCanvas*) const override;
231 SkISize size() const override; 231 SkISize size() const override;
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
420 420
421 class ViaMojo : public Via { 421 class ViaMojo : public Via {
422 public: 422 public:
423 explicit ViaMojo(Sink* sink) : Via(sink) {} 423 explicit ViaMojo(Sink* sink) : Via(sink) {}
424 Error draw(const Src&, SkBitmap*, SkWStream*, SkString*) const override; 424 Error draw(const Src&, SkBitmap*, SkWStream*, SkString*) const override;
425 }; 425 };
426 426
427 } // namespace DM 427 } // namespace DM
428 428
429 #endif//DMSrcSink_DEFINED 429 #endif//DMSrcSink_DEFINED
OLDNEW
« no previous file with comments | « dm/DM.cpp ('k') | dm/DMSrcSink.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698