RFR: 8264293: Create implementation for NSAccessibilityMenu protocol peer

Anton Tarasov ant at openjdk.java.net
Wed Nov 17 10:01:35 UTC 2021


On Tue, 16 Nov 2021 23:53:53 GMT, Alexander Zuev <kizune at openjdk.org> wrote:

> Added implementation for all menu related protocol peers;
> Native methods moved to CommonComponentAccessibility so they are called on correct peers;

Besides what @savoptik mentioned, I left only cosmetics comments. Overall looks ok to me.

src/java.desktop/macosx/native/libawt_lwawt/awt/a11y/CommonComponentAccessibility.m line 1272:

> 1270:     [ThreadUtilities performOnMainThread:@selector(postMenuOpened)
> 1271:         on:(CommonComponentAccessibility *)jlong_to_ptr(element)
> 1272:         withObject:nil waitUntilDone:NO];

Please move `waitUntilDone` to the new line for consistency.

src/java.desktop/macosx/native/libawt_lwawt/awt/a11y/MenuAccessibility.m line 35:

> 33: - (NSAccessibilityRole _Nonnull)accessibilityRole
> 34: {
> 35:         return [[[self parent] javaRole] isEqualToString:@"combobox"]

Please correct the indentation.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6421



More information about the client-libs-dev mailing list