<AWT Dev> OpenJdk11-28-EA JDialog hanging

[Laurent Bourgès]

Hi Martin,

Thanks for your test! Great job :-)
>

Thanks, your fixes are good.


> I applied some minor changes for integration:
>
>  * Renamed to TestSeqEventsMultipleContexts
>   * A bit longer but should describe what this is about
>   * Placed in jdk/java/awt/event/SequencedEvent
>
>  * Encapsulated the window in TestWindow class
>
>  * Instead of waiting 1s for windows to show, loop until windows are shown
> -or the test hard stops-. When I ran in my environment, 1s was not enough.
> This should make the test more resilent. We still depend on a timer but
> that is inevitably.
>
>  * Incremented the time we wait for the TGs to finish. 2s was not enough
> in my runs. However, we now do frequent checks to finish earlier in the
> success path.
>
>  * Calculated expected value in initilization time
>
>  * Changed a bit how the test finishes
>   * System.exit(0) is a failure for jtreg
>   * We need status -1 for jtreg to detect failures, and will do a hard
> stop when time expires
>   * Stack traces for the main thread are not much relevant
>   * dispose window to exit gracefully on the success path
>
>  * Removed unused code (button click action)
>
>  * Added jtreg tags & copyright
>
> Webrev.02 with test integrated:
>
>  * http://cr.openjdk.java.net/~mbalao/webrevs/8204142/8204142.webrev.02
>  *
> http://cr.openjdk.java.net/~mbalao/webrevs/8204142/8204142.webrev.02.zip
>

I tested the new TestSeqEventsMultipleContexts:
- jdk8/9/10: OK
- jdk11: KO
- jdk12+patch: OK

I am not an awt reviewer, but it is OK for me.

Cheers,
Laurent
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20181016/7c61d9fa/attachment.html>