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

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

Issue 2398453002: Rewrap comments to 80 columns in Source/platform/graphics/. (Closed)
Patch Set: Review feedback 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/DecodingImageGenerator.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp b/third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp
index 7b1a30446af637cd679c4f9b4def005758b6c10a..651d8540ada085f6b7a1b6c9337db469b77492c4 100644
--- a/third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp
+++ b/third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp
@@ -78,13 +78,16 @@ bool DecodingImageGenerator::onGetPixels(const SkImageInfo& info,
TRACE_EVENT1("blink", "DecodingImageGenerator::getPixels", "frame index",
static_cast<int>(m_frameIndex));
- // Implementation doesn't support scaling yet so make sure we're not given a different size.
+ // Implementation doesn't support scaling yet, so make sure we're not given a
+ // different size.
if (info.width() != getInfo().width() || info.height() != getInfo().height())
return false;
if (info.colorType() != getInfo().colorType()) {
- // blink::ImageFrame may have changed the owning SkBitmap to kOpaque_SkAlphaType after fully decoding the image frame,
- // so if we see a request for opaque, that is ok even if our initial alpha type was not opaque.
+ // blink::ImageFrame may have changed the owning SkBitmap to
+ // kOpaque_SkAlphaType after fully decoding the image frame, so if we see a
+ // request for opaque, that is ok even if our initial alpha type was not
+ // opaque.
return false;
}
@@ -99,7 +102,8 @@ bool DecodingImageGenerator::onGetPixels(const SkImageInfo& info,
bool DecodingImageGenerator::onQueryYUV8(SkYUVSizeInfo* sizeInfo,
SkYUVColorSpace* colorSpace) const {
- // YUV decoding does not currently support progressive decoding. See comment in ImageFrameGenerator.h.
+ // YUV decoding does not currently support progressive decoding. See comment
+ // in ImageFrameGenerator.h.
if (!m_canYUVDecode || !m_allDataReceived)
return false;
@@ -114,7 +118,8 @@ bool DecodingImageGenerator::onQueryYUV8(SkYUVSizeInfo* sizeInfo,
bool DecodingImageGenerator::onGetYUV8Planes(const SkYUVSizeInfo& sizeInfo,
void* planes[3]) {
- // YUV decoding does not currently support progressive decoding. See comment in ImageFrameGenerator.h.
+ // YUV decoding does not currently support progressive decoding. See comment
+ // in ImageFrameGenerator.h.
ASSERT(m_canYUVDecode && m_allDataReceived);
TRACE_EVENT1("blink", "DecodingImageGenerator::getYUV8Planes", "frame index",
@@ -132,8 +137,9 @@ bool DecodingImageGenerator::onGetYUV8Planes(const SkYUVSizeInfo& sizeInfo,
SkImageGenerator* DecodingImageGenerator::create(SkData* data) {
RefPtr<SegmentReader> segmentReader =
SegmentReader::createFromSkData(sk_ref_sp(data));
- // We just need the size of the image, so we have to temporarily create an ImageDecoder. Since
- // we only need the size, it doesn't really matter about premul or not, or gamma settings.
+ // We just need the size of the image, so we have to temporarily create an
+ // ImageDecoder. Since we only need the size, the premul and gamma settings
+ // don't really matter.
std::unique_ptr<ImageDecoder> decoder = ImageDecoder::create(
segmentReader, true, ImageDecoder::AlphaPremultiplied,
ImageDecoder::GammaAndColorProfileApplied);

Powered by Google App Engine
This is Rietveld 408576698