RFR(XS): 8044495: Remove test demo/jvmti/mtrace/TraceJFrame.java

Mikael Auno mikael.auno at oracle.com
Tue Jun 3 13:43:52 UTC 2014


Thanks for the reviews.

Staffan, could you help me push this? The exported changeset is attached.

Thanks,
Mikael

On 2014-06-02 23:48, serguei.spitsyn at oracle.com wrote:
> Mikael,
> 
> It looks good.
> Thank you for doing this!
> 
> Thanks,
> Serguei
> 
> On 6/2/14 9:14 AM, Mikael Auno wrote:
>> Hi,
>>
>> Could I please have a review of this small fix, removing the test
>> TraceJFrame.java.
>>
>> webrev: http://cr.openjdk.java.net/~miauno/8044495/webrev.00/
>> bug: https://bugs.openjdk.java.net/browse/JDK-8044495
>>
>> This test was created a long, long time ago as it reportedly had been
>> very slow to "step into the first JFrame" in the early stages of JDK
>> 1.5. Thus, it times how long it takes to create that first JFrame while
>> tracing, and then seemingly ignores the result and returns PASS. During
>> its life time, however, we've had lots and lots of failures in this test
>> due to non-existent or misconfigured X11 displays. As such, we believe
>> we're better off without it.
>>
>> Thanks,
>> Mikael
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: JDK-8044495.patch
Type: text/x-patch
Size: 4942 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20140603/ad327ebe/JDK-8044495-0001.patch>


More information about the serviceability-dev mailing list