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

Unified Diff: third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp

Issue 2454123002: Refactor image decoders to use 'colorSpace' instead of 'colorProfile' (Closed)
Patch Set: Created 4 years, 2 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/graphics/DeferredImageDecoder.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp b/third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp
index f537818dff763662ffd2626a2e65aaead7bcadd2..4d57e7b123176fc61ab947c137feacd3d11376a6 100644
--- a/third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp
+++ b/third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp
@@ -180,8 +180,7 @@ bool DeferredImageDecoder::isSizeAvailable() {
}
bool DeferredImageDecoder::hasColorProfile() const {
- return m_actualDecoder ? m_actualDecoder->hasColorProfile()
- : m_hasColorProfile;
+ return m_actualDecoder ? m_actualDecoder->hasColorSpace() : m_hasColorProfile;
f(malita) 2016/10/27 15:57:28 If we're renaming ImageDecoder::hasColorProfile ->
msarett 2016/10/27 16:42:56 Yeah I think so. In general, it probably makes se
}
IntSize DeferredImageDecoder::size() const {
@@ -267,7 +266,7 @@ void DeferredImageDecoder::activateLazyDecoding() {
// future.)
m_canYUVDecode = RuntimeEnabledFeatures::decodeToYUVEnabled() &&
(m_filenameExtension == "jpg");
- m_hasColorProfile = m_actualDecoder->hasColorProfile();
+ m_hasColorProfile = m_actualDecoder->hasColorSpace();
const bool isSingleFrame =
m_actualDecoder->repetitionCount() == cAnimationNone ||

Powered by Google App Engine
This is Rietveld 408576698