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

Unified Diff: third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp

Issue 2660393002: Use gfx::ColorSpace instead of SkColorSpace in Blink (Closed)
Patch Set: Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp
diff --git a/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp b/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp
index 97af0485ac8bf50f46ba0782bb227b80ccee45f1..920587c63a15a4ca4a35953a4ebafb12c4045148 100644
--- a/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp
+++ b/third_party/WebKit/Source/platform/image-decoders/ImageDecoder.cpp
@@ -529,13 +529,15 @@ SkColorSpaceXform* ImageDecoder::colorTransform() {
return nullptr;
}
- if (SkColorSpace::Equals(m_embeddedColorSpace.get(),
- m_colorBehavior.targetColorSpace().get())) {
+ sk_sp<SkColorSpace> dstColorSpace =
+ m_colorBehavior.targetColorSpace().ToSkColorSpace();
+
+ if (SkColorSpace::Equals(m_embeddedColorSpace.get(), dstColorSpace.get())) {
return nullptr;
}
- m_sourceToTargetColorTransform = SkColorSpaceXform::New(
- m_embeddedColorSpace.get(), m_colorBehavior.targetColorSpace().get());
+ m_sourceToTargetColorTransform =
+ SkColorSpaceXform::New(m_embeddedColorSpace.get(), dstColorSpace.get());
return m_sourceToTargetColorTransform.get();
}

Powered by Google App Engine
This is Rietveld 408576698