On Mon, 3 Mar 2025 11:29:33 GMT, Gopal Pattnaik <d...@openjdk.org> wrote:
>> There was no test included with the fix for >> [JDK-8326989](https://bugs.openjdk.org/browse/JDK-8326989), >> >> Hence we are adding a system test now. >> >> Test is written as >> 1. Load html content in web view. >> 2. pick the color of mouse pointer. >> 3. Perform double click. >> 4. pick the color again. >>> expected bahaviour: colour picked in step 2 and 4 should not match. >> >> Verification: >> The test passes with latest webkit source >> Also verified that test fails when the fix for >> [JDK-8326989](https://bugs.openjdk.org/browse/JDK-8326989) is reverted. > > Gopal Pattnaik has updated the pull request incrementally with two additional > commits since the last revision: > > - Addressed Review comments > - Addressed Review comments Looks good with one minor code style issue. tests/system/src/test/java/test/robot/javafx/web/TextSelectionTest.java line 42: > 40: import test.util.Util; > 41: > 42: public class TextSelectionTest extends RobotTestBase{ Minor: please add a space before the `{` ------------- PR Review: https://git.openjdk.org/jfx/pull/1719#pullrequestreview-2657216975 PR Review Comment: https://git.openjdk.org/jfx/pull/1719#discussion_r1979308621
