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

Issue 3027028: Adjust Mac intl2 expectations. (Closed)

Created:
10 years, 5 months ago by chase
Modified:
9 years, 7 months ago
Reviewers:
Nicolas Sylvain
CC:
chromium-reviews, pam+watch_chromium.org
Base URL:
http://src.chromium.org/git/chromium.git
Visibility:
Public.

Description

Adjust Mac intl2 expectations. BUG=50300 TEST=mac intl2 tests are green TBR=nsylvain@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=53748

Patch Set 1 #

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

Messages

Total messages: 2 (0 generated)
chase
10 years, 5 months ago (2010-07-27 02:36:19 UTC) #1
Nicolas Sylvain
10 years, 5 months ago (2010-07-27 03:59:23 UTC) #2
lg

On Mon, Jul 26, 2010 at 7:36 PM, <chase@chromium.org> wrote:

> Reviewers: Nicolas Sylvain,
>
> Description:
> Adjust Mac intl2 expectations.
>
> BUG=50300
> TEST=mac intl2 tests are green
> TBR=nsylvain@chromium.org
>
> Please review this at http://codereview.chromium.org/3027028/show
>
> SVN Base: http://src.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
>
f492ef89c23ab1dd72421ee07dc65eda4d6cc9e5..626746428dbee5a9af18a02a8db10ea0de8ed376
> 100644
> --- a/tools/perf_expectations/perf_expectations.json
> +++ b/tools/perf_expectations/perf_expectations.json
> @@ -12,13 +12,13 @@
>  "linux-release-hardy/startup/warm/t": {"delta": 30, "var": 30},
>  "mac-release-10.5/dhtml/times/t": {"delta": 0, "var": 60},
>  "mac-release-10.5/intl1/times/t": {"delta": 330, "var": 130},
> - "mac-release-10.5/intl2/times/t": {"delta": 400, "var": 120},
> + "mac-release-10.5/intl2/times/t": {"delta": 370, "var": 120},
>  "mac-release-10.5/morejs/times/t": {"delta": -35, "var": 80},
>  "mac-release-10.5/moz/times/t": {"delta": 80, "var": 80},
>  "mac-release-10.5/startup/warm/t": {"delta": 30, "var": 20},
>  "mac-release-10.6/dhtml/times/t": {"delta": -12, "var": 14},
>  "mac-release-10.6/intl1/times/t": {"delta": 480, "var": 110},
> - "mac-release-10.6/intl2/times/t": {"delta": 300, "var": 60},
> + "mac-release-10.6/intl2/times/t": {"delta": 420, "var": 60},
>  "mac-release-10.6/morejs/times/t": {"delta": -40, "var": 50},
>  "mac-release-10.6/moz/times/t": {"delta": 55, "var": 40},
>  "mac-release-10.6/startup/times/t": {"delta": 10, "var": 8},
>
>
>

Powered by Google App Engine
This is Rietveld 408576698