aboutsummaryrefslogtreecommitdiffhomepage
path: root/resources
diff options
context:
space:
mode:
authorGravatar Leon Scroggins III <scroggo@google.com>2016-10-31 14:08:56 -0400
committerGravatar Skia Commit-Bot <skia-commit-bot@chromium.org>2016-10-31 19:23:10 +0000
commitfc49b403eb3353fdefe97672017df10b64dee95a (patch)
tree492ff1afa99917513c071e53ba449c2822797573 /resources
parent7f650bdfd84aa54bf4fb94a2aef89ad931578e8b (diff)
Use correct color table for prior GIF frames
While investigating skbug.com/5883 I noticed that we use the color table for the current frame even while recursively decoding frames that the current frame depends on. This CL updates fCurrColorTable, fCurrColorTableIsReal and fSwizzler before decoding prior frames, and then sets them back afterwards. Move telling the client about the color table into prepareToDecode, since the other callers do not need to do so. (That is only necessary for decoding to index 8, which is unsupported for frames with dependencies.) Add a test that exposes the bug. colorTables.gif has a local color table in its second frame that does not match the global table used by the first frame. GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=4208 Change-Id: Id2dc9e3283adfd92801d2f38726afa74574b1955 Reviewed-on: https://skia-review.googlesource.com/4208 Reviewed-by: Joost Ouwerling <joostouwerling@google.com> Reviewed-by: Matt Sarett <msarett@google.com> Commit-Queue: Leon Scroggins <scroggo@google.com>
Diffstat (limited to 'resources')
-rw-r--r--resources/colorTables.gifbin0 -> 2829 bytes
1 files changed, 0 insertions, 0 deletions
diff --git a/resources/colorTables.gif b/resources/colorTables.gif
new file mode 100644
index 0000000000..f25d13cfaa
--- /dev/null
+++ b/resources/colorTables.gif
Binary files differ