On Tue, 2 May 2023 17:49:39 GMT, Alisen Chung <[email protected]> wrote:

> Adding some DnD tests

test/jdk/java/awt/dnd/DragTriggerEventTest.java line 101:

> 99:         try {
> 100:             Point srcPoint = list.getLocationOnScreen();
> 101:             Rectangle cellBounds = list.getCellBounds(0, 0);

the list is a Swing component that should be accessed on EDT.

test/jdk/java/awt/dnd/DropActionChangedTest.java line 79:

> 77:         } finally {
> 78:             EventQueue.invokeAndWait(() -> {
> 79:                 if(test.frame != null) {

missed space after "if"

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13756#discussion_r1182981771
PR Review Comment: https://git.openjdk.org/jdk/pull/13756#discussion_r1182983382

Reply via email to