On Mon, 9 May 2022 21:05:26 GMT, Phil Race <p...@openjdk.org> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   formatting changes
>
> test/jdk/java/awt/TrayIcon/TrayIconScalingTest.java line 72:
> 
>> 70:         if (!SystemTray.isSupported()) {
>> 71:             System.out.println("SystemTray is not supported");
>> 72:             return;
> 
> Hmm. When you get here, you already displayed the instructions window, and in 
> the jtreg harness, the user will not see this message. They'll just not see 
> the  tray icon and assume it failed.
> I think you should be making use of the "forcePass()" code in PassFailJFrame

Resolved by adding the check as first statement in main()

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

PR: https://git.openjdk.java.net/jdk/pull/8441

Reply via email to