Fix 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

Description was changed from
==========
Fix required frame bug in animated images
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 of frames marked
kDisposeOverwritePrevious. This fixes a bug where frames are drawn on
top of transparent instead of the appropriate prior frame.
TODO: Add a LayoutTest or web platform test.
Bug: 722072
==========
to
==========
Fix 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
==========

Description was changed from
==========
Fix 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
==========
to
==========
Fix 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
==========

Description was changed from
==========
Fix 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
==========
to
==========
Fix 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/+/55b2599c12a1f482bf13f07f855f...
==========