The bug fix looks good to me.
With Regards,
Avik Niyogi
> On 03-Jun-2016, at 11:31 am, Rajeev Chamyal <rajeev.cham...@oracle.com> wrote:
>
>
>
> From: Alexandr Scherbatiy
> Sent: 02 June 2016 12:00
> To: Rajeev Chamyal; Sergey Bylokhov; swing-dev@openjdk.java.net
> <mailto:swing-dev@openjdk.java.net>
> Subject: Re: <Swing Dev> [9] Review request for JDK-[TEST_BUG]
> test/javax/swing/JPopupMenu/8147521/PopupMenuTest.java: compilation failed
>
>
> The fix looks good to me.
>
> Thanks,
> Alexandr.
>
> On 6/2/2016 8:13 AM, Rajeev Chamyal wrote:
> Hello Alexandr,
>
> Please review the updated webrev.
> http://cr.openjdk.java.net/~rchamyal/8158358/webrev.01/
> <http://cr.openjdk.java.net/~rchamyal/8158358/webrev.01/>
>
> Regards,
> Rajeev Chamyal
>
> From: Alexandr Scherbatiy
> Sent: 02 June 2016 01:20
> To: Rajeev Chamyal; Sergey Bylokhov; swing-dev@openjdk.java.net
> <mailto:swing-dev@openjdk.java.net>
> Subject: Re: <Swing Dev> [9] Review request for JDK-[TEST_BUG]
> test/javax/swing/JPopupMenu/8147521/PopupMenuTest.java: compilation failed
>
> On 6/1/2016 10:35 PM, Rajeev Chamyal wrote:
>
>
> Hello All,
>
> Please review the following webrev
>
> Webrev : http://cr.openjdk.java.net/~rchamyal/8158358/webrev.00/
> <http://cr.openjdk.java.net/%7Erchamyal/8158358/webrev.00/>
> Bug : https://bugs.openjdk.java.net/browse/JDK-8158358
> <https://bugs.openjdk.java.net/browse/JDK-8158358>
>
> Fix : Fixed the compilation error.
> The initial summary of the test should be preserved.
>
> Thanks,
> Alexandr.
>
>
>
> Regards,
> Rajeev Chamyal
>
>
>