On 02/02/2015 11:26, Chris Hegarty wrote:
I agree with Alan, addURLStreamHandlerFactory is probably an attractive nuisance. It is not strictly necessary to achieve the goal here; replace the problematic ( with modules ) system property with a service lookup.

For now, I'd like to move this issue forward without the additional new public method. We can have deploy use setURLSHF(), and document the compatibility issue if applets/Webstart apps also try to set a factory. We can revisit this later in 9, if it becomes an issue.

Updated specdiff:
  http://cr.openjdk.java.net/~chegar/8064924/02/specdiff/

I this revision, I omitted the implementation changes, so we can agree the spec changes first. They are much simpler now.
I'm happy with this approach. One outstanding point from the discussion is whether the URLStreamHandlerFactory implementation will need to be granted RuntimePermission("setFactory"), if so then this will need to go into the javadoc.

-Alan.

Reply via email to