<Beans Dev> [9] Review Request: 8136382 SimpleBeanInfo.loadImage succeeds when running with a security manager

[Alan Bateman]

On 09/02/2016 20:49, Sergey Bylokhov wrote:
> On 09.02.16 23:40, Alan Bateman wrote:
>> Sergey - have you consider clarifying the javadoc as part of this 
>> change?
>
> I can add clarification that this is a shortcut to the 
> getResourceAsStream/getResource which simplify loading of images, but 
> the current implementations of getResourceAsStream/getResource change 
> from time to time, so I am not sure that I can describe some stable 
> behavior.
>
With the updated implementation then you are isolated from possible 
changes. You are right that there may be changes on resource 
encapsulation and okay with me if you put all changing the spec until then.

One thing that could be done, separate issue if you want, is update the 
@return to say that it returns null if the resource is not found or the 
resource could not be loaded as an Image. That might be clearer than the 
existing wording.

-Alan.