> When a gif frame's disposal code is DISPOSAL_BGCOLOR and we're maintaining a 
> saved image: we have to flood-fill the frame's bounds. And if the transparent 
> pixel index changes across frames: we could sometimes flood fill the wrong 
> value, resulting in an opaque rectangle in the output image.
> 
> There are a cluster of four related PRs that share the GifComparison class in 
> this PR.
> 
> 1. [8357034](https://github.com/openjdk/jdk/pull/25264)
> 2. ~~[8356137](https://github.com/openjdk/jdk/pull/25044)~~ (integrated)
> 3. [8356320](https://github.com/openjdk/jdk/pull/25076) (this one)
> 4. [8351913](https://github.com/openjdk/jdk/pull/24271)
> 
> The test in this PR required [8356137's 
> PR](https://github.com/openjdk/jdk/pull/25044) to pass.
> 
> This bug can be observed reading this gif animation:
> https://tenor.com/view/fnf-free-download-fnf-free-download-fnf-fd-cyan-gif-15334412416964826224

Jeremy Wood has updated the pull request with a new target base due to a merge 
or a rebase. The pull request now contains 31 commits:

 - Merge branch 'master' into JDK-8356320
   
   # Conflicts:
   #    test/jdk/sun/awt/image/gif/GifBuilder.java
   #    test/jdk/sun/awt/image/gif/GifComparison.java
 - Merge branch 'master' into JDK-8356320
 - 8356320: Use new GifBuilder and remove leo.gif
   
   This is an extension of work for this PR:
   https://github.com/openjdk/jdk/pull/25044#pullrequestreview-2871107750
 - 8356137: adding copyright
 - 8356137: Adding GifBuilder to dynamically create test file
   
   This can be used by multiple gif tests in this directory.
   
   This is in response to:
   https://github.com/openjdk/jdk/pull/25044#pullrequestreview-2871107750
 - 8356137: trivial javadoc update
 - 8356137: only inspect last frame of gif
   
   This makes the main() method much less useful, so I removed it too. (I 
originally used this class to explore a folder of hundreds of gifs to look for 
discrepancies. But the discrepancies were rarely only on the last frame.)
   
   This is in response to:
   https://github.com/openjdk/jdk/pull/25044#discussion_r2109298270
 - 8356137: Adding copyright to GifComparison
 - 8356320: trivial whitespace and comment changes
 - Merge branch 'master' into JDK-8356320
 - ... and 21 more: https://git.openjdk.org/jdk/compare/a86dd56d...3deeb497

-------------

Changes: https://git.openjdk.org/jdk/pull/25076/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=25076&range=02
  Stats: 40 lines in 2 files changed: 40 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/25076.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25076/head:pull/25076

PR: https://git.openjdk.org/jdk/pull/25076

Reply via email to