Hi, Shashi.

A few notes:
 1. All Swing components are still created on non-EDT thread.
 2. Why did you change this line:
      this.setLayout (new BorderLayout ());
    to this:
      frame.setLayout (new BorderLayout ());
    The first line was related to the applet, and actually it was unrelated to 
the frame.
 3. Why did you drop this line?
      Util.waitForCondition
 4. invokeLater() is sued in the "dispose()" method?


On 05/11/2018 08:22, [email protected] wrote:
Hi All, Please review test fix for the below bug.

Bug: https://bugs.openjdk.java.net/browse/JDK-8198624

Webrev: http://cr.openjdk.java.net/~sveerabhadra/8198624/webrev.00/

The problem was the swing components were used in the main thread, causing 
syncing issues between the key press events and swing components. The test has 
been reordered to the proper use of the swing components.

Thanks and regards,

Shashi



--
Best regards, Sergey.

Reply via email to