RFR: 8327478: Add System test to verify TextSelection issue for webkit-617.1 [v3]

Andy Goryachev angorya at openjdk.org
Tue Mar 4 16:22:05 UTC 2025


On Tue, 4 Mar 2025 16:09:03 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> tests/system/src/test/java/test/robot/javafx/web/TextSelectionTest.java line 53:
>> 
>>> 51: 
>>> 52:     private static CountDownLatch webviewLoadLatch = new CountDownLatch(1);
>>> 53:     private Color colorBefore;
>> 
>> suggestion: make `colorBefore/After` _volatile_ since they are being accessed from different threads without any synchronization.
>> I think this should be sufficient (no need for `AtomicReference`).
>
> This isn't strictly needed, since `runAndWait` synchronizes the two threads (although there is no harm in adding volatile).

https://en.wikipedia.org/wiki/Cache_coherence

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1719#discussion_r1979787974


More information about the openjfx-dev mailing list