RFR: 8339341: SurfaceManager cacheMap retains strong references [v4]

[Phil Race]

On Thu, 17 Oct 2024 18:55:45 GMT, Nikita Gubarkov <ngubarkov at openjdk.org> wrote:

>> Ping ! on this .
>
> Because we only use it inside `synchronized` blocks. We need to do this explicitly because in one place it is needed for acquiring two monitors in particular order, and in second place it covers an iteration loop. Therefore there is no need to wrap it into synchronized wrapper.

ok

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20825#discussion_r1805291074