<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">Hi, Dmitriy.<br>
      A few comments:<br>
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      SetMaximizedBounds.java:  please add macosx to the test coverage.
      It does not work now but the fix is under review.<br>
      ChangeGridSize.java/
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      ComponentPreferredSize.java: actionPerformed must be volatile, all
      other constant can be final.<br>
      ModifierRobotKeyTest.java: tempPress must be volatile, access to
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      modifierStatus[] and
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      textStatus should be synchronized somehow.<br>
      <meta http-equiv="content-type" content="text/html;
        charset=ISO-8859-1">
      LockingKeyStateTest.java:I guess we have synchronization problem
      as well.<br>
      <br>
      Additionally, can you dispose all frames in the tests.<br>
      <br>
      On 4/15/14 3:33 PM, Dmitriy Ermashov wrote:<br>
    </div>
    <blockquote cite="mid:534D191E.8030507@oracle.com" type="cite">Hi
      all.
      <br>
      <br>
      Petr, thanks for review!
      <br>
      <br>
      Guys, could you also review this tests?
      <br>
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~yan/8039279/webrev.02/">http://cr.openjdk.java.net/~yan/8039279/webrev.02/</a>
      <br>
      <br>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>