RFR: 8323670: A few client tests intermittently throw ConcurrentModificationException

[Andrey Turbanov]

On Wed, 17 Jan 2024 12:55:52 GMT, Tejesh R <tr at openjdk.org> wrote:

> Suggested fix [JDK-8307091](https://bugs.openjdk.org/browse/JDK-8307091) also created concurrent exception intermittently (monthly once/quarterly once) on CI system. The issue was not able to be reproduced yet, hence proposing an alternative fix which uses iterators to compare the List.
> CI testing shows green.

src/java.desktop/share/classes/javax/swing/plaf/basic/BasicDirectoryModel.java line 417:

> 415:         }
> 416: 
> 417:         private synchronized <T> boolean compareIterators(Iterator<T> iterator1, Iterator<T> iterator2) {

I'm not sure I understand, how this `synchronized` helps to avoid the issue.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17462#discussion_r1460487345