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:

Bug Blocks:

Attachments:

Description	Flags
Does that	tony: review+

Ryosuke Niwa

Reported 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.

Attachments
Does that (12.33 KB, patch) 2012-01-31 13:19 PST, Ryosuke Niwa	tony: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Ryosuke Niwa

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

Tony Chang

Comment 2 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?

Ryosuke Niwa

Comment 3 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.

Ryosuke Niwa

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

Note You need to log in before you can comment on or make changes to this bug.