<AWT Dev> RFR: 8263846: Bad JNI lookup getFocusOwner in accessibility code on Mac OS X [v2]

[Alexander Zuev]

On Sat, 20 Mar 2021 17:06:38 GMT, Phil Race <prr at openjdk.org> wrote:

>> Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixing accessibilityIndexOfChild in the same way
>
> src/java.desktop/macosx/native/libawt_lwawt/awt/JavaComponentAccessibility.m line 802:
> 
>> 800: 
>> 801:     JNIEnv *env = [ThreadUtilities getJNIEnv];
>> 802:     GET_CACCESSIBILITY_CLASS_RETURN(0);
> 
> This isn't necessary. Because the line below takes care of it. The definition looks like
> #define GET_ACCESSIBLEINDEXINPARENT_STATIC_METHOD_RETURN(ret) \
>     GET_CACCESSIBILITY_CLASS_RETURN(ret); \
>     GET_STATIC_METHOD_RETURN(sjm_getAccessibleIndexInParent, sjc_CAccessibility, "getAccessibleIndexInParent", \
>                              "(Ljavax/accessibility/Accessible;Ljava/awt/Component;)I", ret);

Ah, ok, was missed that. Will revert this change.

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

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