<AWT Dev> RFR: 8211267: StackOverflowError happened by TextField.setFont(...) & Add non interactive test case for JDK-8211267

Phil Race philip.race at oracle.com
Wed Dec 12 17:04:28 UTC 2018


That looks better.

-phil.

On 12/12/18 12:59 AM, Ichiroh Takiguchi wrote:
> Hello.
>
> I'm very sorry.
> I forgot to put @test tag into FontChangeTest.java testcase.
> (Thanks, Phil)
>
> Could you review the fix again ?
>
> Bug:    https://bugs.openjdk.java.net/browse/JDK-8211267
> Change: https://cr.openjdk.java.net/~itakiguchi/8211267/webrev.03/
>
> Thanks,
> Ichiroh Takiguchi
>
> On 2018-12-05 09:15, Sergey Bylokhov wrote:
>> Looks fine.
>>
>> On 03/12/2018 00:22, Ichiroh Takiguchi wrote:
>>> Hello.
>>>
>>> Thanks, Sergey.
>>> I created new test case for JDK-8214298.
>>>
>>> I merged JDK-8211267 & JDK-8214298.
>>> I removed interactive test case and added JDK-8214298's code.
>>>
>>> Could you review the fix ?
>>>
>>> Bug:    https://bugs.openjdk.java.net/browse/JDK-8211267
>>> Change: https://cr.openjdk.java.net/~itakiguchi/8211267/webrev.02/
>>>
>>> Thanks,
>>> Ichiroh Takiguchi
>>>
>>> On 2018-12-01 09:35, Sergey Bylokhov wrote:
>>>> Hello,
>>>>
>>>> I think that you can use invokeAndWait instead of invokeLater(),
>>>> which will wait while the code is executed on EDT, and I am not
>>>> sure that you need to interrupt the main thread of the test.
>>>>
>>>> On 26/11/2018 09:19, Ichiroh Takiguchi wrote:
>>>>> Hello.
>>>>> Could you review the fix ?
>>>>>
>>>>> Issue:
>>>>> JDK-8211267 requires non interactive test case
>>>>>
>>>>> Bug:    https://bugs.openjdk.java.net/browse/JDK-8214298
>>>>> Change: https://cr.openjdk.java.net/~itakiguchi/8214298/webrev.00/
>>>>>
>>>>> I'd like to obtain a sponsor for this issue.
>>>>>
>>>>> Thanks,
>>>>> Ichiroh Takiguchi
>>>>> IBM Japan, Ltd.
>>>>>
>>>
>



More information about the awt-dev mailing list