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

2016-03-05 Thread David DeHaven
I've updated the webrev, hopefully one last time with feedback from Joe Darcy. http://cr.openjdk.java.net/~ddehaven/8132743/webrev.2/ - Relocated package Javadoc to above the package declaration in package-info.java - Reworded the Javadoc on the default JSObject ctor A point was brought up that

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

2016-03-05 Thread Mandy Chung
On Mar 5, 2016, at 10:54 AM, David DeHaven wrote: > > > I've updated the webrev, hopefully one last time with feedback from Joe Darcy. > http://cr.openjdk.java.net/~ddehaven/8132743/webrev.2/ > > - Relocated package Javadoc to above the package declaration in > package-info.java > - Reworded

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

2016-03-05 Thread Kevin Rushforth
Looks good. -- Kevin David DeHaven wrote: I've updated the webrev, hopefully one last time with feedback from Joe Darcy. http://cr.openjdk.java.net/~ddehaven/8132743/webrev.2/ - Relocated package Javadoc to above the package declaration in package-info.java - Reworded the Javadoc on the defa