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

Issue 2434243003: Add onIsEncoded to image decoders. (Closed)

Created:
4 years, 2 months ago by cblume
Modified:
4 years, 2 months ago
Reviewers:
bsalomon, vmpstr
CC:
chromium-reviews, krit, drott+blinkwatch_chromium.org, blink-reviews-platform-graphics_chromium.org, dshwang, pdr+graphicswatchlist_chromium.org, jbroman, Justin Novosad, Rik, f(malita), blink-reviews, danakj+watch_chromium.org, ajuma+watch_chromium.org, Stephen Chennney, rwlbuis
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add onIsEncoded to image decoders. We want to be able to query whether an image is encoded or not from within Skia. In order to do this, we are adding onIsEncoded() to the image generators. This is the second part of a 3-part patch. The first is https://chromiumcodereview.appspot.com/2434323002/ The third is https://codereview.chromium.org/2337803002/ BUG=646089

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.h View 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/platform/graphics/DecodingImageGenerator.cpp View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (2 generated)
cblume
PTAL This is the second part in the 3-part patches to add isEncoded to SkImage.
4 years, 2 months ago (2016-10-20 19:09:59 UTC) #2
cblume
4 years, 2 months ago (2016-10-21 07:57:54 UTC) #3
Given the update in https://skia-review.googlesource.com/c/3780/ I think this
patch is no longer needed. I'm closing it.

Powered by Google App Engine
This is Rietveld 408576698