On Thu, 2 Mar 2023 03:42:06 GMT, Sergey Bylokhov <s...@openjdk.org> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Review comment update
>
> src/java.desktop/share/classes/javax/swing/text/html/AccessibleHTML.java line 
> 189:
> 
>> 187:                 
>> rootHTMLAccessibleContext.setElementInfo(rootElementInfo);
>> 188:             }
>> 189:             rootElementInfo.validate();
> 
> probably we should call validate first, so it will be in the valid state 
> before being leaked to the "setElementInfo" method.

I think this can be verified by the test, you can check the number of children 
returned by the a11y API is correct when the doc is replaced.

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

PR: https://git.openjdk.org/jdk/pull/12707

Reply via email to