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.
