<AWT Dev> Reviewer needed - awt/FontClass/CreateFont/DeleteFont test correction

[Coleen Phillimore]

I have to read this bug and reread this thread, but I just backed out 
the change for 6938627 which I believe was the cause of this bug too.  
Hotspot will not use java.io.tmpdir to get a temporary directory, it 
will use the system temporary directory.  There was a JDK change in jdk 
7 that also has to be refixed that Alan's going to do (bug 6950927: 
Testcase failure 
sun/management/jmxremote/bootstrap/JvmstatCountersTest.java)

See bug 7009828: Fix for 6938627 breaks visualvm monitoring when 
-Djava.io.tmpdir is defined

Now, I've read your bug 7003327  - you can filter hsperfdata but it 
won't be there anymore.

Coleen

On 1/12/2011 7:20 PM, Joe Darcy wrote:
> Joe Darcy wrote:
>> On 11/11/2010 4:01 AM, Pavel Tisnovsky wrote:
>>> Phil Race wrote:
>>>> Pavel,
>>>>
>>>> The font implementation and tests are 2D. Not AWT. Don't get thrown by
>>>> package names.
>>> Ah sorry, you are right, of course.
>>>> I was a bit curious about the real reason for this failure.
>>>> I wrote this test a couple of years ago and its never failed before.
>>>> I don't see it failing on any 6-open build.
>>>> I see it failing only in JDK 7 from b90. That seems to be due to 
>>>> HS18b03
>>>> fix
>>>> 6938627: Make temporary directory use property java.io.tmpdir when
>>>> specified
>>> It's failing on my side in OpenJDK 7 and in IcedTea6, because IT6 uses
>>> some backports from OpenJDK 7 (from the log files I can only infer that
>>> this bug first appeared in IcedTea6 when the hs19 were set as default
>>> HotSpot on 2010-10-19). Is it still possible to push the test 
>>> correction
>>> to OpenJDK 6 too, or only to OpenJDK 7?
>>>
>>> Also I'll need a bug id please.
>>>
>>
>> Sorry for the belated reply, please use
>>
>> 7003327 Correct test awt/FontClass/CreateFont/DeleteFont
>>
>> Approved for OpenJDK 6.
>>
>> -Joe
>
> Hello.
>
> If this can be pushed in the next day or two, I'm open to having this 
> bug fix in b21.
>
> Thanks,
>
> -Joe