RFR: 8319925: CSS.BackgroundImage incorrectly uses double-checked locking
Prasanta Sadhukhan
psadhukhan at openjdk.org
Tue Dec 5 07:59:19 UTC 2023
On Mon, 4 Dec 2023 21:51:15 GMT, Sergey Bylokhov <serb 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)
Yes, seems like setBase can be used to change image URL on the fly...Modified to use image field for DCL...
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16917#issuecomment-1840192317
More information about the client-libs-dev
mailing list