<Swing Dev> [8] Request for approval for CR 7105119 - [TEST_BUG] [macosx] In test UIDefaults.toString() mast be called with the invokeLater()

Konstantin Shefov konstantin.shefov at oracle.com
Wed Feb 13 13:30:02 UTC 2013


On 13-Feb-13 15:52, Alexander Scherbatiy wrote:
> On 2/12/2013 6:37 PM, Konstantin Shefov wrote:
>> Hello,
>>
>> Please review a fix for the issue:
>>
>> 7105119 - [TEST_BUG] [macosx] In test UIDefaults.toString() mast be 
>> called with the invokeLater()
>>
>> The webrev is http://cr.openjdk.java.net/~kshefov/7105119/webrev.00/
>
>    Could you give more information why the UIDefaults.toString() needs 
> to be called with the invokeLater()?
>    The UIDefaults.toString()  has explicit synchronized keyword and 
> the UIManager.getDefaults() method uses classLock object for 
> synchronization.

Indeed.

I do not know, why this test hanged on mac, most probably jtreg issue. 
Without jtreg it passed ok. With invokeLater() it passed with JTREG also.

Anyway, I have suggest to convert this test to JTREG main test, because 
the parent bug 4300666 does not depend on applet execution:
http://cr.openjdk.java.net/~kshefov/7105119/webrev.01

After conversion, the test works fine on Mac and other OSs.

-Konstantin

>
>    Thanks,
>    Alexandr.
>
>>
>> Thanks,
>> Konstantin
>




More information about the swing-dev mailing list