On Thu, 21 Mar 2024 16:51:36 GMT, Damon Nguyen <dngu...@openjdk.org> wrote:
>> Convert
>> java/awt/image/multiresolution/MultiDisplayTest/MultiDisplayTest.java applet
>> test to main using PassFailJFrame
>
> Damon Nguyen has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Review comments
Marked as reviewed by azvegint (Reviewer).
test/jdk/java/awt/image/multiresolution/MultiDisplayTest.java line 109:
> 107: frame = new JFrame("MultiDisplayTest");
> 108: frame.setLayout(new BorderLayout());
> 109: Button b = new Button("Start");
Just noticed that the `frame` is not used in test itself, so I guess we can get
rid of it, and make this method return the "Start" `JButton`, and use it with
`splitUIBottom`/`splitUIRight` instead of `testUI`
-------------
PR Review: https://git.openjdk.org/jdk/pull/18354#pullrequestreview-1952846634
PR Review Comment: https://git.openjdk.org/jdk/pull/18354#discussion_r1534301518
