Bug 98305 - Simplify the magnitude-perf test harness
Summary: Simplify the magnitude-perf test harness
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ojan Vafai
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-03 14:50 PDT by Ojan Vafai
Modified: 2012-10-03 15:59 PDT (History)
5 users (show)

See Also:

Attachments
Patch (12.53 KB, patch)
2012-10-03 15:34 PDT, Ojan Vafai 		rniwa: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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>