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

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

Issue 2523943002: Explicitly specify target color space to ImageDecoder at creation (Closed)
Patch Set: Rebase Created 4 years, 1 month 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/DeferredImageDecoderTestWoPlatform.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTestWoPlatform.cpp b/third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTestWoPlatform.cpp
index dba110eb13cbe1a03086c8cc8bffdff9d7b1b979..578d79800ca756b587d0534d25a97983e47756e1 100644
--- a/third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTestWoPlatform.cpp
+++ b/third_party/WebKit/Source/platform/graphics/DeferredImageDecoderTestWoPlatform.cpp
@@ -40,7 +40,7 @@ static void mixImages(const char* fileName,
SharedBuffer::create(file->data(), bytesForFirstFrame);
std::unique_ptr<DeferredImageDecoder> decoder = DeferredImageDecoder::create(
partialFile, false, ImageDecoder::AlphaPremultiplied,
- ImageDecoder::ColorSpaceIgnored);
+ ImageDecoder::ColorSpaceIgnored, nullptr);
ASSERT_NE(decoder, nullptr);
sk_sp<SkImage> partialImage = decoder->createFrameAtIndex(0);
@@ -101,7 +101,7 @@ TEST(DeferredImageDecoderTestWoPlatform, fragmentedSignature) {
EXPECT_FALSE(ImageDecoder::hasSufficientDataToSniffImageType(*buffer));
EXPECT_EQ(nullptr, DeferredImageDecoder::create(
buffer, false, ImageDecoder::AlphaPremultiplied,
- ImageDecoder::ColorSpaceIgnored));
+ ImageDecoder::ColorSpaceIgnored, nullptr));
// Append the rest of the data. We should be able to sniff the signature
// now, even if segmented.
@@ -110,7 +110,7 @@ TEST(DeferredImageDecoderTestWoPlatform, fragmentedSignature) {
std::unique_ptr<DeferredImageDecoder> decoder =
DeferredImageDecoder::create(buffer, false,
ImageDecoder::AlphaPremultiplied,
- ImageDecoder::ColorSpaceIgnored);
+ ImageDecoder::ColorSpaceIgnored, nullptr);
ASSERT_NE(decoder, nullptr);
EXPECT_TRUE(String(testFiles[i]).endsWith(decoder->filenameExtension()));
}

Powered by Google App Engine
This is Rietveld 408576698