On Mon, 14 Mar 2022 06:29:30 GMT, Srinivas Mandalika <smandal...@openjdk.org> 
wrote:

>> Create a test for JDK-4715496
>> 
>> Issue observed in the above bug is that the 
>> JTable.AccessibleJTableCell.getAccessible name returns the cell instance 
>> string instead of the cell text.
>> The test is being added to verify the same.
>> This review is for migrating tests from a closed test suite to open.
>
> Srinivas Mandalika has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   frame dispose call on EDT

test/jdk/javax/accessibility/8283015/AccessibleJTableCellNameTest.java line 28:

> 26:  * @key headful
> 27:  * @bug 4715496
> 28:  * @summary AccessibleJTableCell.getAccessible name incorrectly returns 
> cell instance string instead of cell text.

Please split the long lines, use the 80 chars per line.

test/jdk/javax/accessibility/8283015/AccessibleJTableCellNameTest.java line 60:

> 58: 
> 59:             for (int i = 0; i <= colNames.length - 1; i++) {
> 60:                 Accessible accessible = 
> jTable.getAccessibleContext().getAccessibleTable()

I remember a similar code in some other test which probably was integrated 
already. But I just found that the jtable here is a Swing component that should 
be accessed in the EDT.

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

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

Reply via email to