Hi,<br><br><div class="gmail_quote"><blockquote style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;" class="gmail_quote">we had a short offline discussion with Oleg, and we both agree there is no need to clear the event queue explicitly. If the interrupt() call comes from AppContext.dispose(), a new EDT will not be initialized (because of the check for AppContext.isDisposed() in initDispatchThread()).<br>
</blockquote><br>Thanks, I updated both versions of the patch with the comment suggested by Dave (however my englisch is not perfect, so please feel free to modify it as you like):<br><a href="http://cr.openjdk.java.net/~ceisserer/7081670/webrev_full.03/">http://cr.openjdk.java.net/~ceisserer/7081670/webrev_full.03/</a><br>
<a href="http://cr.openjdk.java.net/~ceisserer/7081670/webrev_minimal.01/">http://cr.openjdk.java.net/~ceisserer/7081670/webrev_minimal.01/</a><br><br>If the &quot;full&quot; version is considered too risky for JDK7, it would be great if the minimal could make it in, as caciocavallo-web gets bitten by this bug.<br>
The &quot;minimal&quot; version has a more extensive comment, as for the &quot;minimal&quot; version the check for isHeldByCurrentThread is not just cosmetic but required.<br><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

So your &quot;full&quot; version of the fix looks fine. As a sanity check, please run all the tests from test/java/awt/EQ and test/java/awt/EDT with your changes, it shouldn&#39;t take much time.<br></blockquote><div><br>
I ran the tests, but EventDispatchThread/LoopRobustness fails with:<br><blockquote style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;" class="gmail_quote">Caused by: java.lang.RuntimeException: Die, AWT-Event Queue thread!<br>
    at HostileCrasher.&lt;clinit&gt;(LoopRobustness.java:157)<br>    ... 23 more<br>Exception in thread &quot;main&quot; java.lang.RuntimeException: Test FAILED: second thread hasn&#39;t notified MainThread<br>    at LoopRobustness.main(LoopRobustness.java:70)<br>
</blockquote><div><br><br> and HandleExceptionOnEDT seems to time out somehow:<br><br></div><blockquote style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;" class="gmail_quote">
Exception in thread &quot;main&quot; sun.awt.SunToolkit$OperationTimedOut: 10005<br>    at sun.awt.X11.XToolkit.syncNativeQueue(XToolkit.java:2443)<br>    at sun.awt.SunToolkit.realSync(SunToolkit.java:1580)<br>    at sun.awt.SunToolkit.realSync(SunToolkit.java:1512)<br>
    at test.java.awt.regtesthelpers.Util.waitForIdle(Util.java:184)<br>    at HandleExceptionOnEDT.main(HandleExceptionOnEDT.java:75)<br></blockquote><br>However, I get exatly the same behaviour with the proprietary release build of JDK7,<br>
so I guess those tests have been broken before. Are those tests executed before release?<br><br>Thanks, Clemens<br></div></div>