Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(4)

Issue 1567053002: Animated PNG implementation

Created:
4 years, 9 months ago by MaxStepin
Modified:
3 years, 8 months ago
Reviewers:
CC:
chromium-reviews, shans, rjwright, blink-reviews-animation_chromium.org, darktears, blink-reviews, kinuko+watch, Eric Willigers
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Animated PNG implementation BUG=1171 BUG=437662

Patch Set 1 #

Patch Set 2 : size_t fixes #

Patch Set 3 : Update due to recent changes in original PNGImageDecoder #

Patch Set 4 : Update due to recent SharedBuffer->SegmentReader changes #

Patch Set 5 : Update due to changes in original code #

Patch Set 6 : Update due to changes in original code #

Patch Set 7 : Update due to recent Blink Reformat #

Patch Set 8 : Update due to recent SkColorSpace changes #

Patch Set 9 : Moved some code to PNGImageReader.cpp and PNGImageReader.h #

Messages

Total messages: 9 (5 generated)
acterhd
Somebody review this patch.
4 years, 3 months ago (2016-07-08 08:56:51 UTC) #5
Noel Gordon
@Max. I've been OOO and noted that chrome has updated to 1.6.X libpng while I ...
3 years, 10 months ago (2016-12-01 04:20:02 UTC) #6
MaxStepin
On 2016/12/01 04:20:02, noel gordon wrote: > @Max. I've been OOO and noted that chrome ...
3 years, 10 months ago (2016-12-01 08:58:33 UTC) #7
Noel Gordon
3 years, 10 months ago (2016-12-01 10:09:44 UTC) #8
On 2016/12/01 08:58:33, MaxStepin wrote:

> I don't know what OOO means, but OK.

OOO === out of the office,  I was away from work on sabbatical for many months.

> libpng version doesn't matter.

OK good.

> My patch is still fine, but recently I learned that there is another patch in
> the works:
> https://codereview.chromium.org/2386453003/

IC.

Powered by Google App Engine
This is Rietveld 408576698