Bug 98305

Summary:

Simplify the magnitude-perf test harness

Product:

WebKit

Reporter:

Ojan Vafai <ojan>

Component:

New Bugs

Assignee:

Ojan Vafai <ojan>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

abarth, eric, pdr, rniwa, tony

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	rniwa: review+

Description Ojan Vafai 2012-10-03 14:50:00 PDT

Simplify the magnitude-perf test harness

Comment 1 Ojan Vafai 2012-10-03 15:34:13 PDT

Created attachment 166973 [details]
Patch

Comment 2 Ryosuke Niwa 2012-10-03 15:51:05 PDT

Comment on attachment 166973 [details]
Patch

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

> LayoutTests/ChangeLog:11
> +        -Remove the chromium.Interval stuff. This API no longer exists.
> +        -Reduce the number of iterations to 8.
> +        -Start magnitude at 1 instead of 0.
> +        -Remove all the iterationsPerRun junk. This complexity was not achieving the goal of

Nit: insert a space after "-".

Comment 3 Ojan Vafai 2012-10-03 15:59:03 PDT

Committed r130337: <http://trac.webkit.org/changeset/130337>