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

Ichiroh Takiguchi takiguc at linux.vnet.ibm.com
Wed Dec 12 08:59:34 UTC 2018


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