I also believe with use of invokeAndWait(), we probably do not need waitUntilShown(frame) too.

On 15-Nov-19 1:04 PM, Prasanta Sadhukhan wrote:

Hi Jay,

I guess we should also do the dispose of the JFrame via try-finally in main() for the cleanup.

Regards

Prasanta

On 15-Nov-19 12:50 PM, Jayathirth D V wrote:

Hi Prasanta,

waitUntilShown() inside createAndShowGUI() was doing the same job, but its good if we use InvokeAndWait().

Please find updated webrev:

http://cr.openjdk.java.net/~jdv/8234184/webrev.01/

Latest webrev is green in internal CI test system.

Thanks,

Jay

*From:*Prasanta Sadhukhan
*Sent:* Thursday, November 14, 2019 11:00 PM
*To:* Jayathirth D V <jayathirth....@oracle.com>; awt-dev@openjdk.java.net *Subject:* Re: <AWT Dev> [14] RFR JDK-8234184:[TESTBUG] java/awt/Mouse/EnterExitEvents/ModalDialogEnterExitEventsTest.java fails in Windows

Probably we can call invokeAndWait() for createAndShowGUI instead of invokeLater for frame to be shown, which may be more cleaner approach.

Regards

Prasanta

On 14-Nov-19 10:46 PM, Jayathirth D V wrote:

    Hello All,

    Please review the following fix in JDK14:

    Bug : https://bugs.openjdk.java.net/browse/JDK-8234184

    Webrev : http://cr.openjdk.java.net/~jdv/8234184/webrev.00/

    Issue : We are disposing the frame even before we record Mouse
    Entry event.

    Solution : Added appropriate waits to record Mouse entry event.

    Test fails every time without fix in Internal CI and passes
    everytime with fix.

    Thanks,

    Jay

Reply via email to