Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(346)

Issue 3750016: Adjust expectations for vistadual/morejs. (Closed)

Created:
10 years, 2 months ago by chase
Modified:
9 years, 7 months ago
Reviewers:
Nicolas Sylvain
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Adjust expectations for vistadual/morejs. BUG=none TEST=vistadual/morejs is green TBR=nsylvain@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=62917

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M tools/perf_expectations/perf_expectations.json View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
chase
10 years, 2 months ago (2010-10-18 12:45:41 UTC) #1
Nicolas Sylvain
10 years, 2 months ago (2010-10-18 16:20:13 UTC) #2
lg

On Mon, Oct 18, 2010 at 5:45 AM, <chase@chromium.org> wrote:

> Reviewers: Nicolas Sylvain,
>
> Description:
> Adjust expectations for vistadual/morejs.
>
> BUG=none
> TEST=vistadual/morejs is green
> TBR=nsylvain@chromium.org
>
> Please review this at http://codereview.chromium.org/3750016/show
>
> SVN Base: http://git.chromium.org/git/chromium.git
>
> Affected files:
>  M tools/perf_expectations/perf_expectations.json
>
>
> Index: tools/perf_expectations/perf_expectations.json
> diff --git a/tools/perf_expectations/perf_expectations.json
> b/tools/perf_expectations/perf_expectations.json
> index
>
2462ed80bb94f98589b16345466e55dee76a5b7e..5604389085af56a510022bc16d241ecd778bd53a
> 100644
> --- a/tools/perf_expectations/perf_expectations.json
> +++ b/tools/perf_expectations/perf_expectations.json
> @@ -48,7 +48,7 @@
>  "vista-release-dual-core/dhtml/times/t": {"improve": -22, "regress": 45},
>  "vista-release-dual-core/intl1/times/t": {"improve": 180, "regress": 370},
>  "vista-release-dual-core/intl2/times/t": {"improve": -180, "regress":
> 215},
> - "vista-release-dual-core/morejs/times/t": {"improve": -160, "regress":
> -75},
> + "vista-release-dual-core/morejs/times/t": {"improve": -130, "regress":
> -60},
>  "vista-release-dual-core/moz/times/t": {"improve": -3, "regress": 133},
>  "vista-release-dual-core/startup/times/t": {"improve": -15, "regress":
> 15},
>  "vista-release-dual-core/v8_benchmark/score/score": {"improve": 1650,
> "regress": 1070},
>
>
>

Powered by Google App Engine
This is Rietveld 408576698