Re: [RFR JDK-8017187] [TEST BUG] [macosx] After click "test", the case failed automatically with thrown exception in the log since jdk8b75

[nadeesh tv]

Hi all,
Thanks for the review.
Regards,
Nadeesh
On 8/18/2015 7:09 PM, Sergey Bylokhov wrote:

Looks fine.

On 17.08.15 14:05, nadeesh tv wrote:

Hi all,
Gentle reminder
Regards,
Nadeesh
On 8/12/2015 1:33 PM, Alexander Scherbatiy wrote:

 The fix looks good to me.

  Thanks,
  Alexandr.

On 8/12/2015 10:46 AM, nadeesh tv wrote:

Hi,

Please review the updated webrev links
open- http://cr.openjdk.java.net/~sgupta/8017187/webrev.02/
closed- http://cr.openjdk.java.net/~sgupta/8017187/webrev.03/
Thanks and Regards,
Nadeesh TV
On 8/11/2015 4:54 PM, nadeesh tv wrote:

Hi,

Please review the updated webrev links
closed - http://cr.openjdk.java.net/~sgupta/8017187/webrev.01/
open - http://cr.openjdk.java.net/~sgupta/8017187/webrev.00/
Thanks and Regards,
Nadeesh TV


On 8/6/2015 8:13 PM, nadeesh tv wrote:

Hi,
Please review the updated webrev links
closed - http://cr.openjdk.java.net/~sgupta/8017187/webrev.01/
open - http://cr.openjdk.java.net/~sgupta/8017187/webrev.00/

Thanks and Regards,
Nadeesh TV
On 8/5/2015 9:04 PM, Alexander Scherbatiy wrote:

On 8/5/2015 5:05 PM, nadeesh tv wrote:

HI all,
Please review the updated webrev links
closed - http://cr.openjdk.java.net/~sgupta/8017187/webrev.00/
open - http://cr.openjdk.java.net/~sgupta/8017187/webrev.01/

- the patch does not contain the html file as it expected but it still is shown on the provided open link. It seems that the index.html file has not been updated. - the swing calls like "!menu.isSelected()" should be invoked on EDT.

  Thanks,
  Alexandr.

Thanks and Regards,
Nadeesh TV

On 7/29/2015 3:38 PM, Alexander Scherbatiy wrote:

- It looks like the html file has been used only because the test was manual and it can be simply removed. - Is it possible to use a robot to press keys rather than put KeyEvents directly to the SystemEventQueue?

 Thanks,
 Alexandr.


On 7/24/2015 9:13 PM, nadeesh tv wrote:

Hi all,

Please review a test bug fix

TEST : *closed/javax/swing/JMenu/4213634/bug4213634.java *
BUG ID -*https://bugs.openjdk.java.net/browse/JDK-8017187*

Please*move  the test from closed repo tom open repo*.

The webrev is:http://cr.openjdk.java.net/~sgupta/8017187/webrev.00/ add to open repo

http://cr.openjdk.java.net/~kshefov/8017187/webrev.diff/ - diff with previous version of the closed test.

--
Thanks and Regards,
Nadeesh TV

--
Thanks and Regards,
Nadeesh TV