Looks good! Thanks for cleaning this up. /Staffan
On 2 jun 2014, at 18:14, Mikael Auno <mikael.a...@oracle.com> 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