<AWT Dev> [7u8] Review request for 7189350: Fix failed for CR 7162144

[Oleg Pekhovskiy]

Looks fine to me.

Thanks,
Oleg

20.08.2012 19:24, Anthony Petrov wrote:
> Hello,
>
> Please review a fix for 
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7189350 at:
>
> http://cr.openjdk.java.net/~anthony/7u8-1-missingAWTThread-7189350.0/
>
> With this fix we resolve an issue originally described in 7162144 by 
> properly handling thread interruption requests. We no longer override 
> the Thread.interrupt() for the EDT, and hence don't clash with 
> NetBeans using thread interruption for its own purposes.
>
> Note that there's still not a regression test since NetBeans' test is 
> very heavy (over 100MB) and impractical for our purposes. However, the 
> NetBeans team has tested a developer build with this fix and found no 
> regressions.
>
> PS. We aren't yet sure if we want the same fix for JDK 8, hence we fix 
> it for 7u8 first.
>
> -- 
> best regards,
> Anthony