RFR: 8361286: Allow enabling of background loading for images loaded from an InputStream [v2]
John Hendrikx
jhendrikx at openjdk.org
Wed Aug 20 13:49:49 UTC 2025
On Tue, 19 Aug 2025 20:49:58 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> John Hendrikx has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Add since tags
>
> modules/javafx.graphics/src/main/java/javafx/scene/image/Image.java line 685:
>
>> 683: * using the specified parameters.
>> 684: * <p>
>> 685: * If loading in the background is requested, then the progress property can
>
> suggestion (also in L734, L776):
>
>
> * If loading in the background is requested, then the {@link #progressProperty() progress} property can
Added this for the 4 constructors that have the `backgroundLoading` parameter. Thanks :)
> modules/javafx.graphics/src/main/java/javafx/scene/image/Image.java line 1136:
>
>> 1134: private AsyncOperation constructPeer() {
>> 1135: if(inputSource == null) {
>> 1136: return loadImageAsync(
>
> minor: wrapping is not necessary in this method
Fixed
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1875#discussion_r2288227454
PR Review Comment: https://git.openjdk.org/jfx/pull/1875#discussion_r2288225837
More information about the openjfx-dev
mailing list