On 8/6/2015 5:43 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/
Could you recreate the webrev and put it with index.html file?
Thanks,
Alexandr.
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
