On Wed, 18 Nov 2020 19:03:17 GMT, Alexander Zuev <kiz...@openjdk.org> wrote:
>> 8256108: Create implementation for NSAccessibilityElement protocol peer > > Alexander Zuev has updated the pull request incrementally with one additional > commit since the last revision: > > Fixed the year in file header. > > Added comments on a reason for the common component layer existance. src/java.desktop/macosx/native/libawt_lwawt/awt/JavaComponentAccessibility.m line 140: > 138: // Please see > https://developer.apple.com/documentation/appkit/nsaccessibilityprotocol > 139: // for more details. > 140: @interface CommonComponentAccessibility : JavaComponentAccessibility > <NSAccessibilityElement> { I remember we had a plan to implement this step by step, replacing the parts of the old API with some new parts. If the plan still in place then why we do not implement `NSAccessibilityElement` in the `JavaComponentAccessibility ` class, so the usage of current (old) methods like `accessibilityPositionAttribute/accessibilitySizeAttribute/accessibilityFocusedAttribute` could be replaced by the new methods from the NSAccessibilityElement protocol like accessibilityParent/accessibilityFrame/isAccessibilityFocused. If we plan to implement it from scratch then why the CommonComponentAccessibility extends JavaComponentAccessibility? It is not obvious from the change since the `accessibilityFrame()` is implemented from scratch mostly duplicate `accessibilityPositionAttribute+accessibilitySizeAttribute`, but the `accessibilityParent()` uses the code from the `JavaComponentAccessibility ` ------------- PR: https://git.openjdk.java.net/jdk/pull/1290