On Mon, 30 Sep 2024 14:09:43 GMT, Alexey Ivanov <[email protected]> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   removed redundant frame.setLocation()
>
> test/jdk/ProblemList.txt line 468:
> 
>> 466: java/awt/Dialog/MakeWindowAlwaysOnTop/MakeWindowAlwaysOnTop.java 
>> 8266243 macosx-aarch64
>> 467: java/awt/dnd/BadSerializationTest/BadSerializationTest.java 8277817 
>> linux-x64,windows-x64
>> 468: java/awt/dnd/DragSourceMotionListenerTest.java 8225131 windows-all
> 
> `DragSourceMotionListenerTest` passes for me on Windows. Does it need to be 
> problem-listed?

It passes on windows. I'm checking with the updated changes and running it 
multiple times to check for any intermittent issues.

> test/jdk/ProblemList.txt line 469:
> 
>> 467: java/awt/dnd/BadSerializationTest/BadSerializationTest.java 8277817 
>> linux-x64,windows-x64
>> 468: java/awt/dnd/DragSourceMotionListenerTest.java 8225131 windows-all
>> 469: java/awt/dnd/RejectDragTest.java 7124259 macosx-all
> 
> I wonder if `RejectDragTest` still fails on macOS. Latest comment in 
> [JDK-7124259](https://bugs.openjdk.org/browse/JDK-7124259) is dated February 
> 2020. The problem could be fixed by now.

Missed seeing this one among the review comments. It still fails on macOS.

> test/jdk/ProblemList.txt line 470:
> 
>> 468: java/awt/dnd/DragSourceMotionListenerTest.java 8225131 windows-all
>> 469: java/awt/dnd/RejectDragTest.java 7124259 macosx-all
>> 470: java/awt/dnd/DnDHTMLToOutlookTest/DnDHTMLToOutlookTest.java 8027424 
>> generic-all
> 
> DnDHTMLToOutlookTest passes for me with Mozilla Thunderbird 128.2.3 as well 
> as with Microsoft Word and Outlook version 2302 on Windows.
> 
> I haven't tested on Linux or macOS.

DnDHTMLToOutlookTest DnD => Thunderbird failed on linux for @azvegint . So I 
kept it problemlisted to investigate it later. This test was originally 
problemlisted on all platforms due to DnD HTML copy over to Thunderbird.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21213#discussion_r1786638530
PR Review Comment: https://git.openjdk.org/jdk/pull/21213#discussion_r1786638195
PR Review Comment: https://git.openjdk.org/jdk/pull/21213#discussion_r1786638056

Reply via email to