Sorry, please ignore my previous mail. 
My mail client synced up mails (after re-formatting) and showed this as a new 
mail.
Sorry for the confusion.

Thanks,
Krishna

> On 31-Jan-2019, at 12:30 PM, Krishna Addepalli <krishna.addepa...@oracle.com> 
> wrote:
> 
> Hi Shashi,
> I presume you meant this fix to be reviewed for 13.
> 
> Could you re-upload the patch with latest code? I tried importing the patch, 
> but it fails to apply on the latest code.
> Here is the output I get:
> 
> applying 
> http://cr.openjdk.java.net/~sveerabhadra/8061359/webrev.00/jdk11-dev-client-1.patch
> patching file 
> src/java.desktop/macosx/classes/sun/lwawt/macosx/CAccessible.java
> Hunk #1 FAILED at 0
> Hunk #2 FAILED at 32
> Hunk #3 FAILED at 109
> Hunk #4 succeeded at 173 with fuzz 2 (offset 14 lines).
> 3 out of 4 hunks FAILED -- saving rejects to file 
> src/java.desktop/macosx/classes/sun/lwawt/macosx/CAccessible.java.rej
> abort: patch failed to apply
> 
> Thanks,
> Krishna
> 
>> On 01-Oct-2018, at 3:10 PM, shashidhara.veerabhadra...@oracle.com 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 type as shown in the attached picture. 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
>> 
>> <IMG_5218.JPG>
> 

Reply via email to