Re: [11] Review Request: 8205537 Drop of sun.applet package

2018-07-02 Thread Sergey Bylokhov
I can argue that this is not p2 as related JDK-8201446, but it is definitely not p4. On 02/07/2018 09:30, Phil Race wrote: It is not necessary and I reject that it is a P3. Let's leave it for 12. -phil. On 07/02/2018 09:24 AM, Sergey Bylokhov wrote: I think it is a good thing to drop this

Re: [11] Review Request: 8205537 Drop of sun.applet package

2018-07-02 Thread Phil Race
It is not necessary and I reject that it is a P3. Let's leave it for 12. -phil. On 07/02/2018 09:24 AM, Sergey Bylokhov wrote: I think it is a good thing to drop this api completely in the same release as JDK-8201446. This is not a p4 and so it can be be fixed and integrated in jdk11 if the

Re: [11] Review Request: 8205537 Drop of sun.applet package

2018-07-02 Thread Sergey Bylokhov
I think it is a good thing to drop this api completely in the same release as JDK-8201446. This is not a p4 and so it can be be fixed and integrated in jdk11 if the fix is ready. On 02/07/2018 09:10, Phil Race wrote: Sergey, I think at this point, this is not a P3 for 11. So I won't approve

Re: [11] Review Request: 8205537 Drop of sun.applet package

2018-07-02 Thread Phil Race
Sergey, I think at this point, this is not a P3 for 11. So I won't approve it for 11, but will consider it for 12. -phil. On 7/2/2018 6:36 AM, Sergey Bylokhov wrote: Hello. Please review the fix for jdk11. Bug: https://bugs.openjdk.java.net/browse/JDK-8205537 Webrev:

[11] Review Request: 8205537 Drop of sun.applet package

2018-07-02 Thread Sergey Bylokhov
Hello. Please review the fix for jdk11. Bug: https://bugs.openjdk.java.net/browse/JDK-8205537 Webrev: http://cr.openjdk.java.net/~serb/8205537/webrev.00/ sun.applet is an internal package contained some code related to implementation of applets and appletviewer. Some of its classes were