RFR: 8319925: CSS.BackgroundImage incorrectly uses double-checked locking
Sergey Bylokhov
serb at openjdk.org
Mon Dec 4 21:53:36 UTC 2023
On Mon, 4 Dec 2023 21:06:04 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
> > It is probably easy just drop the usage of loadedImage and use the image instead?
>
> Ideally. Yet there's a corner case: if `url` is null, there's nothing to load; and it doesn't make sense to re-try, the URL won't change. If it were not for this case, the `image` field alone would do the job.
Not sure how it is expected to work, the url is a parameter and can be changed on the fly(see the usage of base in the StyleSheet.java)
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16917#issuecomment-1839542420
More information about the client-libs-dev
mailing list