The fix looks good to me.

Thanks,
Alexandr.

On 6/9/2016 8:58 PM, Daniil Titov wrote:
Thank you, Mandy!

The long line is corrected.

Webrev: http://cr.openjdk.java.net/~dtitov/8156960/jdk/webrev.03
         http://cr.openjdk.java.net/~dtitov/8156960/webrev.03

Bug: https://bugs.openjdk.java.net/browse/JDK-8156960

Best regards,
Daniil




-----Original Message-----
From: Mandy Chung
Sent: Thursday, June 09, 2016 9:23 AM
To: Daniil Titov
Cc: Erik Joelsson; David Dehaven; Stuart Marks; build-dev; 
build-infa-...@openjdk.java.net; awt-dev; Kevin Rushforth
Subject: Re: Review Request: 8156960 Deprecate JSObject.getWindow(Applet) method


On Jun 9, 2016, at 9:08 AM, Daniil Titov <daniil.x.ti...@oracle.com> wrote:

Thank you, Erik!

Please review the new version of the patch that has "../" fixed:

Webrev: http://cr.openjdk.java.net/~dtitov/8156960/jdk/webrev.02
        http://cr.openjdk.java.net/~dtitov/8156960/webrev.02/

Looks okay.  Nit: in JSObject.java - line 154 long line to be wrapped.

Mandy

Reply via email to