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

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

Issue 2706453003: switch to new ImageGenerator API (Closed)
Patch Set: Created 3 years, 10 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 f242a1d11733dbba50aefe10c742a6c6ea13acce..bcef98152cf919758d3a96d6a55a0e5cd6841ef8 100644
--- a/third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp
+++ b/third_party/WebKit/Source/platform/graphics/DeferredImageDecoder.cpp
@@ -328,11 +328,11 @@ sk_sp<SkImage> DeferredImageDecoder::createFrameImageAtIndex(
knownToBeOpaque ? kOpaque_SkAlphaType : kPremul_SkAlphaType,
m_colorSpaceForSkImages);
- DecodingImageGenerator* generator = new DecodingImageGenerator(
+ std::unique_ptr<DecodingImageGenerator> generator(new DecodingImageGenerator(
f(malita) 2017/02/17 16:41:08 nit: WTF::makeUnique(...);
reed1 2017/02/17 18:19:35 Done.
m_frameGenerator, info, std::move(segmentReader), m_allDataReceived,
- index, m_frameData[index].m_uniqueID);
- sk_sp<SkImage> image = SkImage::MakeFromGenerator(
- generator); // SkImage takes ownership of the generator.
+ index, m_frameData[index].m_uniqueID));
+ generator->setCanYUVDecode(m_canYUVDecode);
+ sk_sp<SkImage> image = SkImage::MakeFromGenerator(std::move(generator));
if (!image)
return nullptr;
@@ -346,8 +346,6 @@ sk_sp<SkImage> DeferredImageDecoder::createFrameImageAtIndex(
m_frameData[index].m_uniqueID = image->uniqueID();
}
- generator->setCanYUVDecode(m_canYUVDecode);
-
return image;
}

Powered by Google App Engine
This is Rietveld 408576698