Re: RFR: 8196440: Regression automated Test 'java/awt/TrayIcon/PopupMenuLeakTest/PopupMenuLeakTest.java' fails

2021-10-24 Thread Prasanta Sadhukhan
On Fri, 22 Oct 2021 19:59:31 GMT, Alexander Zvegintsev wrote: > The test doesn't take into account that SystemTray may not be supported. > In case of Ubuntu this happens when `_NET_SYSTEM_TRAY` is not present. Marked as reviewed by psadhukhan (Reviewer). - PR: https://git.openjdk.

Re: RFR: 8273704: DrawStringWithInfiniteXform.java failed : drawString with InfiniteXform transform takes long time

2021-10-24 Thread Prasanta Sadhukhan
On Fri, 22 Oct 2021 19:25:28 GMT, Phil Race wrote: > This test has only failed once - for reasons that are not clear. > I wanted to just increase the timeout but the test was written such that it > always waited as long > as the timeout since it had an un-canceled scheduled task. > > So I've al

Re: RFR: 8202932: java/awt/Component/NativeInLightShow/NativeInLightShow.java fails [v3]

2021-10-24 Thread Sergey Bylokhov
On Fri, 22 Oct 2021 12:05:24 GMT, Alexander Zvegintsev wrote: >> Looks like the test does not fail anymore. >> >> However some cleanup was made: >> >> - fixed wildcard imports >> - frame centered and explicitly disposed at the end >> - removed unused `setAutoDelay()` call (no key/mouse key eve

Re: RFR: 8273704: DrawStringWithInfiniteXform.java failed : drawString with InfiniteXform transform takes long time

2021-10-24 Thread Sergey Bylokhov
On Fri, 22 Oct 2021 19:25:28 GMT, Phil Race wrote: > This test has only failed once - for reasons that are not clear. > I wanted to just increase the timeout but the test was written such that it > always waited as long > as the timeout since it had an un-canceled scheduled task. > > So I've al

Re: RFR: 8275170: Some jtreg sound tests should be marked headful

2021-10-24 Thread Sergey Bylokhov
On Sun, 24 Oct 2021 16:45:15 GMT, Phil Race wrote: > This makes the tests eligible to be run on headful systems. In practice it > seems like ONLY headful systems actually have the sound devices. The sound > keyword isn't understood by anything yet, so adding headful is the only way > we have r

Re: RFR: 8275170: Some jtreg sound tests should be marked headful

2021-10-24 Thread Phil Race
On Fri, 22 Oct 2021 19:01:27 GMT, Phil Race wrote: > This fix adds "headful sound" keywords to a number of javax/sound jtreg tests > > The jtreg javax.sound tests are written in such a way that if a needed audio > device > or other platform resource is not available, they just exit with a "pass