<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