Bug 77469

Summary:

PerfTestRunner should automatically create pre#log

Product:

WebKit

Reporter:

Ryosuke Niwa <rniwa>

Component:

Tools / Tests

Assignee:

Ryosuke Niwa <rniwa>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, arv, eric, morrita, ojan

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Bug Depends on:

77074

Bug Blocks:

77037

Attachments:

Description	Flags
Does that	tony: review+

Description Ryosuke Niwa 2012-01-31 13:09:30 PST

It's silly for each test file to have <pre id="log"></pre>. PerfTestRunner should automatically create that.

Comment 1 Ryosuke Niwa 2012-01-31 13:19:13 PST

Created attachment 124807 [details]
Does that

Comment 2 Tony Chang 2012-01-31 13:30:06 PST

Comment on attachment 124807 [details]
Does that

View in context: https://bugs.webkit.org/attachment.cgi?id=124807&action=review

> PerformanceTests/resources/runner.js:60
>      this.printStatistics(statistics, this.log);

Remove this.log?

Comment 3 Ryosuke Niwa 2012-01-31 13:34:07 PST

Thanks for the review.

(In reply to comment #2)
> (From update of attachment 124807 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=124807&action=review
> 
> > PerformanceTests/resources/runner.js:60
> >      this.printStatistics(statistics, this.log);
> 
> Remove this.log?

Oops, will fix and land.

Comment 4 Ryosuke Niwa 2012-01-31 13:35:56 PST

Committed r106386: <http://trac.webkit.org/changeset/106386>