Hi Peter!
Thank you for reviewing this!
--Best regards,
Daniil
On 9/29/2014 12:00 PM, Pete Brunet wrote:
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
