On 1/19/20 7:34 pm, Sergey Bylokhov wrote:
On 1/13/20 10:49 pm, Sureshkumar Mahaliswamy wrote:
The LookandFeel parameter is normally set as part of the test execution as VM 
arguments,
hence not looping through all the available LAF.

This parameter is rarely set, actually never set when the tests are executed
via mach5, I suggest to iterate over the list of L&F in the test since we
know that its behavior can depend on them.

Or probably we need to try to load the image again because of the spec of the 
method state this:
"Loads the image, returning only when the image is loaded."



Kindly review and let me know if there are any comments.

JBS:https://bugs.openjdk.java.net/browse/JDK-8235900

Webrev: http://cr.openjdk.java.net/~nnigam/8235900/webrev.1/
Testing: tested with jtreg on mac and other platforms.
Thanks,
Suresh.


-----Original Message-----
From: Sergey Bylokhov
Sent: Monday, December 23, 2019 12:43 AM
To: Sureshkumar Mahaliswamy <sureshkumar.mahalisw...@oracle.com>; 
swing-dev@openjdk.java.net
Subject: Re: <Swing Dev> RFR : [15] : 8235900 : PopupMenu Opaque property is 
not reflecting the Parents property on MAC OS

Hi, Sureshkumar.

Probably it will be better to do something instead of completely disabling the 
test. The problem exists in Aqua, but we can test some other L&F on macOS.
I guess the best solution is to iterate over all installed L&fs, but exclude 
Aqua.

On 12/17/19 11:55 am, Sureshkumar Mahaliswamy wrote:
Hi All,

Kindly review the small patch.

Added the annotation @requires(os.family!=mac) to ignore the test execution on 
MAC,since Aqua LaF forces all popups to be heavyweight.

Marking this test not to be executed for MAC platform.

JBS:https://bugs.openjdk.java.net/browse/JDK-8235900

Webrev: http://cr.openjdk.java.net/~nnigam/8235900/webrev.0/

Testing: tested with jtreg.

Thanks,

Suresh.



--
Best regards, Sergey.





--
Best regards, Sergey.

Reply via email to