Hi Daniil, I reviewed the two webrevs original change: http://cr.openjdk.java.net/~dtitov/8015376/webrev.00 backout: http://cr.openjdk.java.net/~dtitov/8059136/webrev.00/
and saw no problems with the backout of the original change. Pete On 9/26/14, 3:48 PM, Daniil Titov wrote: > Hi, > > Please review webrev that reverses changes done in the team forest > (https://bugs.openjdk.java.net/browse/JDK-8015376 removal of applet > demos). > > JDK-8015376 changes were integrated in 8u-dev team forest only, so > there in nothing to reverse in JDK 9. > > Both changesets will have to come through 8u-dev forest. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8059136 > 8u40 webrev: http://cr.openjdk.java.net/~dtitov/8059136/webrev.00 > <http://cr.openjdk.java.net/%7Edtitov/8059136/webrev.00> > > Thanks! > Daniil > > > -------- Original Message -------- > Subject: Re: Request for Approval: 8015376 - Remove jnlp and > applet files from the JDK samples > Date: Thu, 25 Sep 2014 10:01:17 +0100 > From: Seán Coffey <sean.cof...@oracle.com> > To: daniil titov <daniil.x.ti...@oracle.com>, > jdk8u-...@openjdk.java.net > > > > Please add a suitable noreg- label to bug report. (noreg-build I guess) > > Approved. > > regards, > Sean. > > On 25/09/2014 00:48, daniil titov wrote: >> Please approve the changes that remove applets from JDK samples. 8u40 >> changes were already integrated in the team forest. They were reviewed >> by the team members at time of integration. JDK 9 changes are ready >> for integration as well. >> >> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8015376 >> JDK 8u40 webrev: http://cr.openjdk.java.net/~dtitov/8015376/webrev.00 >> <http://cr.openjdk.java.net/%7Edtitov/8015376/webrev.00> >> JDK 9 webrev: http://cr.openjdk.java.net/~dtitov/8015376.9/webrev.00 >> <http://cr.openjdk.java.net/%7Edtitov/8015376.9/webrev.00> >> >> Thanks, >> Daniil > > >
