RFR: 8146918: ConcurrentModificationException in MediaPlayer [v2]

[Kevin Rushforth]

On Sun, 21 Apr 2024 14:39:46 GMT, n-gabe <duke at openjdk.org> wrote:

>> There is a ConcurrentModificationException in MediaPlayer when removing a MediaView from it. The root cause is that you can't iterate over a HashSet with for (WeakReference<MediaView> vref : viewRefs) and removing items from the collection by viewRefs.remove(vref); within this loop.
>
> n-gabe has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update Copyright

modules/javafx.media/src/main/java/javafx/scene/media/MediaPlayer.java line 2:

> 1: /*
> 2:  * Copyright (c) 2010, 2022, 2024, Oracle and/or its affiliates. All rights reserved.

Copyright lines must have exactly one or two years. Please remove `, 2022`

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1377#discussion_r1574796815