<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    +1<br>
    <br>
    Minor nit, there's a wild card import in the test :-<br>
    <pre>  24 import java.awt.*;

-phil.
</pre>
    <br>
    <br>
    <div class="moz-cite-prefix">On 10/11/18 11:52 AM, Sergey Bylokhov
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:409aae2f-5a04-faa3-f9d8-2a0f247d5983@oracle.com">Hello.
      <br>
      Please review the fix for jdk 12.
      <br>
      <br>
      Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8207070">https://bugs.openjdk.java.net/browse/JDK-8207070</a>
      <br>
      Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8207070/webrev.01">http://cr.openjdk.java.net/~serb/8207070/webrev.01</a>
      <br>
      <br>
      Bug description:
      <br>
      This is one more bug which was caused by the removing the main
      appcontext in case of applets/webstart application. Since then the
      toolkit thread has no appcontext, and we should not call any
      methods which use appcontext on the toolkit thread.
      <br>
      <br>
      In the updated code we have tried to call
      "EventQueue.invokeLater()". Note that in case of applets/webstart
      we have a few EDT, so we need to know which EDT(stored per
      appcontext) should be used, but since toolkit thread has no
      appconext we throw NPE.
      <br>
      <br>
      Fix description:
      <br>
       - WWindowPeer.java: the updateGC() always will be called on the
      EDT related to the peer's target component, regardless on what
      thread the WWindowPeer.displayChanged() was called.
      <br>
       - WWtoolkit.java: in case of applet/webstart we will run the
      GraphicsEnvironment.displayChanged() on the separate thread. Note
      that it is possible to use the toolkit thread for this, except for
      D3D which use the toolkit thread for rendering. It is possible to
      workaround it, but I do not want to push it to jdk8, I will get
      rid of this in the later release.
      <br>
      <br>
      <br>
    </blockquote>
    <br>
  </body>
</html>