Review request: 8044063: Remove com.sun.java.browser.* from jdk repo

2014-06-18 Thread Mandy Chung
com.sun.java.browser.dom and com.sun.java.browser.net classes are legacy and only used by old plugin. Time to remove them. Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8044063/webrev.00/ https://bugs.openjdk.java.net/browse/JDK-8044063 Jon - having a second thought and decide to c

Re: Review request: 8044063: Remove com.sun.java.browser.* from jdk repo

2014-06-18 Thread Alan Bateman
On 18/06/2014 19:22, Mandy Chung wrote: com.sun.java.browser.dom and com.sun.java.browser.net classes are legacy and only used by old plugin. Time to remove them. Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8044063/webrev.00/ Good to see this being removed, the changes looks good

Re: Review request: 8044063: Remove com.sun.java.browser.* from jdk repo

2014-06-18 Thread Joe Darcy
On 06/18/2014 11:27 AM, Alan Bateman wrote: On 18/06/2014 19:22, Mandy Chung wrote: com.sun.java.browser.dom and com.sun.java.browser.net classes are legacy and only used by old plugin. Time to remove them. Webrev: http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8044063/webrev.00/ Good to se

Re: Review request: 8044063: Remove com.sun.java.browser.* from jdk repo

2014-06-20 Thread Mandy Chung
Thanks for the review. FYI. I'm going to push this change to jdk9/client as this depends on the deploy change that is not integrated (just anxious to get this out of my queue rather than waiting it to show up in jdk9/dev). Mandy On 6/18/14 11:32 AM, Joe Darcy wrote: On 06/18/2014 11:27 AM,