On Fri, 29 Nov 2024 05:15:24 GMT, Abhishek Kumar <[email protected]> wrote:
>> Damon Nguyen has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - String comparison
>> - Review comments
>
> test/jdk/java/awt/Focus/UnaccessibleChoice/AccessibleChoiceTest.java line 110:
>
>> 108: loc = button.getLocationOnScreen();
>> 109: });
>> 110: robot.mouseMove(loc.x + button.getWidth() / 2, loc.y
>
> get button's width and height on EDT along with location.
Done
> test/jdk/java/awt/Focus/UnaccessibleChoice/AccessibleChoiceTest.java line 116:
>
>> 114: robot.mousePress(InputEvent.BUTTON1_DOWN_MASK);
>> 115: robot.mouseRelease(InputEvent.BUTTON1_DOWN_MASK);
>> 116: System.out.println("EIGHT");
>
> Any reason to add these many print statements ? Is it for debugging purpose?
Yes, was trying to test failure points again. Removed now!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22333#discussion_r1866309901
PR Review Comment: https://git.openjdk.org/jdk/pull/22333#discussion_r1866309559
