On Sat, 31 Aug 2024 00:30:50 GMT, Harshitha Onkar <[email protected]> wrote:

> Since the following are on unchanged lines, I'm including it as a 
> consolidated note:
> 
> 1. robot.setAutoWaitForIdle(true);
>    robot.setAutoDelay(200);
>    doesn't change anything as there are no robot mouse or key events so it 
> can be removed.
> 2. Individual `robot.delay()` can be added after UI creation and when frame 
> state is changed to stabilize the test.
> 3. Frame is assigned null after disposing which is not required and can be 
> removed.
> 
> ```
>     private void disposeFrame() {
>         if (frame != null) {
>             frame.dispose();
>             frame = null;
>         }
>     }
> ```

Fixed all these.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/20777#issuecomment-2322837108

Reply via email to