Hi, Petr.
The fix looks good.
On 7/23/14 5:14 PM, Petr Pchelko wrote:
Hello, AWT Team.
This is a reminder.
Could you please review the backport.
Thank you.
With best regards. Petr.
On 15 июля 2014 г., at 16:16, Anthony Petrov <[email protected]> wrote:
+1
--
best regards,
Anthony
On 7/15/2014 2:01 PM, Petr Pchelko wrote:
Hello, AWT team.
Please review the fix for the issue:
https://bugs.openjdk.java.net/browse/JDK-8048549
The 8u version is available here:
http://cr.openjdk.java.net/~pchelko/9/8048549/webrev.8u/
For reference, the 9 version is available here:
http://cr.openjdk.java.net/~pchelko/9/8048549/webrev/
The 8u version is different, so a new review is needed. In 9 we've moved AWT
initialization from awt.m to LWCToolkit.m, so we did not need to use
ThreadUtilities.
In 8u we need to use ThreadUtilities to figure out if AWT is running embedded
or not. All the rest of the fix is the same as in 9.
Thank you.
With best regards. Petr.
--
Best regards, Sergey.
