> This PR adds a title-mentioned fallback to `ImageStorage.java` and a test for > that specific scenario. > > In `ImageStorage.loadAll()` we still prefer the Image path that was provided > by the user, but if it is missing we will now try to add a `@1x` suffix to > Image name and give it one last try. > > Added test includes a binary file `check...@1x.png`. This was added to > differentiate it from the collection of `checker.png` files and their higher > scale variants, but it is a direct copy of already existing `checker.png` > test resource, just with a different name and suffix. Test should fail > without a change in `ImageStorage.java` and pass with it.
Lukasz Kostyra has updated the pull request incrementally with one additional commit since the last revision: Call createInputStream again when fallback fails ------------- Changes: - all: https://git.openjdk.org/jfx/pull/1598/files - new: https://git.openjdk.org/jfx/pull/1598/files/9e46931e..96155157 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=1598&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1598&range=01-02 Stats: 4 lines in 1 file changed: 2 ins; 0 del; 2 mod Patch: https://git.openjdk.org/jfx/pull/1598.diff Fetch: git fetch https://git.openjdk.org/jfx.git pull/1598/head:pull/1598 PR: https://git.openjdk.org/jfx/pull/1598