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

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Wed Feb 13 13:32:09 UTC 2013


13.02.2013 17:30, Konstantin Shefov wrote:
> 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.
I suggest to investigate it and file the new CR against jtreg.
> 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.
Fix looks good.
>
> -Konstantin
>
>>
>>    Thanks,
>>    Alexandr.
>>
>>>
>>> Thanks,
>>> Konstantin
>>
>


-- 
Best regards, Sergey.




More information about the swing-dev mailing list