On Tue, 24 Mar 2026 22:13:42 GMT, Jeremy Wood <[email protected]> wrote:

> This fixes a bug in CommonComponentAccessibility.m: we were passing an 
> AccessibleContext, when we should have been passing in an AccessibleAction.
> 
> This happened to work because many AccessibleContexts also implement 
> AccessibleAction, but if the AccessibleAction was a separate object then this 
> triggered a IncompatibleClassChangeError

test/jdk/javax/accessibility/8380849/AccessibleActionAsSeparateClassTest.java 
line 88:

> 86:         protected void paintComponent(Graphics g) {
> 87:             g.setColor(clickedSuccessfully ? Color.green : Color.gray);
> 88:             g.fillOval(0, 0, getWidth(),getHeight());

Suggestion:

            g.fillOval(0, 0, getWidth(), getHeight());

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30418#discussion_r2990783902

Reply via email to