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

Side by Side Diff: dm/DMSrcSink.h

Issue 1952063002: Create SkColorSpaceXform to handle color conversions (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Rebase 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 194 matching lines...) Expand 10 before | Expand all | Expand 10 after
205 // Mimic legacy behavior and apply no color correction. 205 // Mimic legacy behavior and apply no color correction.
206 kBaseline_Mode, 206 kBaseline_Mode,
207 207
208 // Color correct images into a specific dst color space. If you happen to have this 208 // Color correct images into a specific dst color space. If you happen to have this
209 // monitor, you're in luck! The unmarked outputs of this test should di splay 209 // monitor, you're in luck! The unmarked outputs of this test should di splay
210 // correctly on this monitor in the Chrome browser. If not, it's useful to know 210 // correctly on this monitor in the Chrome browser. If not, it's useful to know
211 // that this monitor has a profile that is fairly similar to Adobe RGB. 211 // that this monitor has a profile that is fairly similar to Adobe RGB.
212 // TODO (msarett): Should we add a new test with a new monitor and verif y that outputs 212 // TODO (msarett): Should we add a new test with a new monitor and verif y that outputs
213 // look identical on two different dsts? 213 // look identical on two different dsts?
214 kDst_HPZR30w_Mode, 214 kDst_HPZR30w_Mode,
215
216 #if !defined(GOOGLE3)
217 // Use QCMS for color correction.
218 kQCMS_HPZR30w_Mode,
219 #endif
215 }; 220 };
216 221
217 ColorCodecSrc(Path, Mode, sk_sp<SkColorSpace>); 222 ColorCodecSrc(Path, Mode);
218 223
219 Error draw(SkCanvas*) const override; 224 Error draw(SkCanvas*) const override;
220 SkISize size() const override; 225 SkISize size() const override;
221 Name name() const override; 226 Name name() const override;
222 bool veto(SinkFlags) const override; 227 bool veto(SinkFlags) const override;
223 private: 228 private:
224 Path fPath; 229 Path fPath;
225 Mode fMode; 230 Mode fMode;
226 sk_sp<SkColorSpace> fDstSpace;
227 }; 231 };
228 232
229 class SKPSrc : public Src { 233 class SKPSrc : public Src {
230 public: 234 public:
231 explicit SKPSrc(Path path); 235 explicit SKPSrc(Path path);
232 236
233 Error draw(SkCanvas*) const override; 237 Error draw(SkCanvas*) const override;
234 SkISize size() const override; 238 SkISize size() const override;
235 Name name() const override; 239 Name name() const override;
236 private: 240 private:
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
392 396
393 class ViaMojo : public Via { 397 class ViaMojo : public Via {
394 public: 398 public:
395 explicit ViaMojo(Sink* sink) : Via(sink) {} 399 explicit ViaMojo(Sink* sink) : Via(sink) {}
396 Error draw(const Src&, SkBitmap*, SkWStream*, SkString*) const override; 400 Error draw(const Src&, SkBitmap*, SkWStream*, SkString*) const override;
397 }; 401 };
398 402
399 } // namespace DM 403 } // namespace DM
400 404
401 #endif//DMSrcSink_DEFINED 405 #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