Re: < Swing Dev> [9] Review Request:JDK-8130481 [TEST_BUG] javax/swing/JPopupMenu/6583251/bug6583251.java failed with UnsupportedOperation exception

[pooja chopra]

Hi Alexandr,

Please review updated webrev link :-

http://cr.openjdk.java.net/~pchopra/8130481/webrev.06/

Regards,
Pooja
On 8/5/2015 2:46 PM, Alexander Scherbatiy wrote:

On 8/5/2015 10:31 AM, pooja chopra wrote:

Hi Alexandr,

Please review  update webrev  link :-

The webrev is  : http://cr.openjdk.java.net/~pchopra/8130481/webrev.03/

    This looks better. There are just minor issues:
    - run swing components on EDT like: menu.show(frame, 0, 0)

- format the code in 'if' block or check the mercurial settings in .hgrc file like diff = -w // ignore white space when comparing lines

  Thanks,
  Alexandr.

Regards,
Pooja
On 7/29/2015 4:04 PM, Alexander Scherbatiy wrote:

On 7/29/2015 9:12 AM, pooja chopra wrote:

Hello Sergey ,
Please review  update webrev link below :-

The webrev is : http://cr.openjdk.java.net/~pchopra/8130481/webrev.02/

After the SystemTray.isSupported() is checked it is not expected that the UnsupportedOperationException is thrown. The test should fail in this case.

  Thanks,
  Alexandr.

Regards,
Pooja
On 7/16/2015 12:32 AM, Sergey Bylokhov wrote:

Hello,

I suggest to check support of systemTray at the beginning of the test.

On 15.07.15 13:07, pooja chopra wrote:

Hello ,

Corrected the webrev link below . Please review.

Regards,
Pooja
On 7/15/2015 3:26 PM, pooja chopra wrote:

Hello,

Please review a fix for issue :-

8130481 [TEST_BUG] javax/swing/JPopupMenu/6583251/bug6583251.java failed with UnsupportedOperation exception

Test bug fix.
https://bugs.openjdk.java.net/browse/JDK-8130481

The webrev is : http://cr.openjdk.java.net/~pchopra/8130481/webrev.01/

Regards,
Pooja