This resolves the NPE reported in 8395930, and it enhances the preexisting test for JDK-8381236 to cover this case.
The failing use case involves an Accessible object where `axObj.getAccessibleContext() == null`. Having a `null` AccessibleContext is an atypical case, but it is explicitly identified/allowed in the contract for `Accessible.getAccessibleContext()`. Prior to [8381236 changes](https://github.com/openjdk/jdk/pull/30578/changes), calling CAccessible.getCAccessible(ax) would also fail with a NPE. So in this PR I'm proposing preserving that behavior (and documenting it), and moving the null check to 8381236's new `disposeRecursively` method. --------- - [x] I confirm that I make this contribution in accordance with the [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). ------------- Commit messages: - 8395930: removing unrelated change - 8395930: formatting line break - 8395930: apply null check in disposeRecursively - Revert "8395930: avoid NPE for null axContext" - 8395930: fixing typo - 8395930: avoid NPE for null axContext - 8395930: add failing test for "Unreadable Label" - Merge branch 'master' of https://github.com/mickleness/jdk - Merge branch 'openjdk:master' into master - 8385367: stop using stale/orphaned AccessibleContext Changes: https://git.openjdk.org/jdk/pull/33657/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=33657&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8395930 Stats: 26 lines in 2 files changed: 18 ins; 0 del; 8 mod Patch: https://git.openjdk.org/jdk/pull/33657.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/33657/head:pull/33657 PR: https://git.openjdk.org/jdk/pull/33657
