[9-dev] RfR: JDK-8132743: Move netscape.javascript package from jdk.plugin to new module

David DeHaven david.dehaven at oracle.com
Thu Mar 3 22:55:35 UTC 2016


>> JBS Issue:
>> https://bugs.openjdk.java.net/browse/JDK-8132743
>> 
>> Code review:
>> http://cr.openjdk.java.net/~ddehaven/8132743/webrev.0/
> 
> Looks okay.  There is no @since - I guess it’s okay because netscape.javascript has been shipped with plugin for long time.

I can't track down when it was first included. It pre-dates anything I've looked at so far.


> package-info.java
>    "when running in an {@link java.applet.Applet Applet}” - is this true when running with JavaFX webkit?

Yes, I believe so, assuming you have a JSObject representing the root window object. Maybe that should be reworded, I think just remove the "when running in an Applet" part.


> JSObject.java
>   @throws JSException is missing in the methods
> 
>   Does it throw NPE if the parameter is null?  Or JSException - that needs to be specified.
> 
>   Nit: it’d be good to wrap null with {@code null} in the javadoc.

Ok. I can fix that.

-DrD-



More information about the jdk9-dev mailing list