>> 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-
