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

Issue 2793493002: Add paint invalidation tests for APNG images (Closed)

Created:
3 years, 8 months ago by Noel Gordon
Modified:
3 years, 8 months ago
Reviewers:
chrishtr
CC:
darktears, blink-reviews, blink-reviews-animation_chromium.org, chromium-reviews, Eric Willigers, rjwright, shans
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Add paint invalidation tests for APNG images Add layout tests for paint invalidation of an animated PNG (APNG) image. Add APNG image resource derived from the GIF resource. Add invalidation tests for the basic image case, when it's background CSS, and when it's off-screen. BUG=707073 Review-Url: https://codereview.chromium.org/2793493002 Cr-Commit-Position: refs/heads/master@{#461010} Committed: https://chromium.googlesource.com/chromium/src/+/e3a4f13335fb6b0e38226f395c993888556b1617

Patch Set 1 #

Messages

Total messages: 14 (9 generated)
Noel Gordon
PTAL.
3 years, 8 months ago (2017-03-31 01:55:05 UTC) #4
chrishtr
lgtm Cool.
3 years, 8 months ago (2017-03-31 02:12:49 UTC) #6
Noel Gordon
Thanks, submitting.
3 years, 8 months ago (2017-03-31 03:18:29 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2793493002/1
3 years, 8 months ago (2017-03-31 03:18:54 UTC) #11
commit-bot: I haz the power
3 years, 8 months ago (2017-03-31 03:26:20 UTC) #14
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e3a4f13335fb6b0e38226f395c99...

Powered by Google App Engine
This is Rietveld 408576698