DescriptionFix required frame bug in APNGs
If frame |i|'s disposal method is kDisposeOverwritePrevious, frame
|i+1| does not necessarily share |i|'s required frame. If |i| is
independent (e.g. by filling the screen and being opaque or using
kBlendAtopBgcolor), |i+1| may still depend on |i-1|.
When looking for the required frame, skip over frames marked
kDisposeOverwritePrevious. This fixes a bug where frames are drawn on
top of transparent instead of the appropriate prior frame.
Add a LayoutTest that exercises the problem.
BUG=722072
Review-Url: https://codereview.chromium.org/2887123003
Cr-Commit-Position: refs/heads/master@{#472897}
Committed: https://chromium.googlesource.com/chromium/src/+/55b2599c12a1f482bf13f07f855f6caf643af390
Patch Set 1 #Patch Set 2 : Add a LayoutTest #
Total comments: 2
Patch Set 3 : Move comment #
Messages
Total messages: 16 (9 generated)
|