<Swing Dev> <AWT Dev> [15] RFR : 8249278 : Revert JDK-8226253 which breaks the spec of AccessibleState.SHOWING for JList

[Ambarish Rapte]

Hello Phil,

 

Thanks for the review and correcting the mail group,

Please take a look at the updated webrev with C years also backed out: 

http://cr.openjdk.java.net/~arapte/a11y/8249278/webrev.1

 

Regards,

Ambarish.

From: Philip Race 
Sent: Wednesday, July 15, 2020 2:00 AM
To: Ambarish Rapte <ambarish.rapte at oracle.com>; swing-dev at openjdk.java.net
Cc: awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> [15] RFR : 8249278 : Revert JDK-8226253

 

adding swing-dev which is the correct list for this Swing change.

It looks like a faithful backout of
https://bugs.openjdk.java.net/browse/JDK-8226253
https://hg.openjdk.java.net/jdk/jdk/rev/3ea8a0c5c264

except that the (c) year is remaining 2020 .. which I think just
indicates the silliness of all these (c) updates since the only
changes to these files in 2020 were those which you have now backed out.

So I think you should revert the (c) changes as well.

If some one else comes along and updates all files in JDK with updates in 2020
and so re-adds it then that is their problem ...

-phil.

On 7/14/20, 5:37 AM, Ambarish Rapte wrote: 

Hi,

 

Please review this fix: 

JBS: https://bugs.openjdk.java.net/browse/JDK-8249278

Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Earapte/a11y/8249278/webrev.0/"http://cr.openjdk.java.net/~arapte/a11y/8249278/webrev.0/

 

 

This change reverts the fix for JDK-8226253.

The fix of JDK-8226253 breaks the specification of AccessibleState.SHOWING for JList and has caused a regression with JTree.

There is no alternative to fix JDK-8226253 on JDK side and the issue should be really fixed on Screen reader side.

So in order to keep the correctness of spec and avoid any new regression, the fix needs to be reverted.

 

 

Regards,

Ambarish
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/swing-dev/attachments/20200714/ace9f5ca/attachment.htm>