<AWT Dev> [14] RFR JDK-8234184:[TESTBUG] java/awt/Mouse/EnterExitEvents/ModalDialogEnterExitEventsTest.java fails in Windows
Prasanta Sadhukhan
prasanta.sadhukhan at oracle.com
Fri Nov 15 07:48:55 UTC 2019
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.d.v at oracle.com>;
>> awt-dev at 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
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/awt-dev/attachments/20191115/d00b81fa/attachment-0001.html>
More information about the awt-dev
mailing list