Looks fine.
On 01/10/2018 21:54, shashidhara.veerabhadra...@oracle.com wrote:
Hi Sergey, Thanks for your review and here is the updated webrev:
http://cr.openjdk.java.net/~sveerabhadra/8061359/webrev.01/
Thanks and regards,
Shashi
On 02/10/18 1:14 AM, Sergey Bylokhov wrote:
Hi, Shashi.
It looks like this line is not necessary;
"if(ptr != 0) {"
The prt is checked at the beginning of this method.
On 01/10/2018 02:54, Shashidhara Veerabhadraiah wrote:
Hi All, Please review a fix for an accessibility bug.
Bug: https://bugs.openjdk.java.net/browse/JDK-8061359
Webrev: http://cr.openjdk.java.net/~sveerabhadra/8061359/webrev.00/
Problem: The checkbox component state changes thro' keyboard space
key were not getting voiced over. The problem was that those
events(state property changes) were not processed and hence just
ignored. Now added conditions to send changes for the state changes
associated with checkbox component.
To test this, please use the swingset2 demo(or any swing application
demo) and go to any checkbox control(JCheckBox) and select and
deselect it by pressing space key. One should hear the component
state, component name and component. Please make sure you have turned
on the voice over on mac os(Command + F5).
Currently I am not sure to add any manual test to test this out.
Please let me know if such a test needs to be added.
Thanks and regards,
Shashi
--
Best regards, Sergey.
