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

Issue 3121018: Fix redness in the tree by silencing all the perf tests that are failing.... (Closed)

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

Description

Fix redness in the tree by silencing all the perf tests that are failing. Most of the errors look like it came from the V8 merge a while back, but the perf expectations were a little bit too strict. 2 new perf regressions, both fairly small: xp perf intl2: http://build.chromium.org/buildbot/perf/xp-release-dual-core/intl2/report.html?history=140&rev=55952 Blame: 55671 (ananta) xp perf intl1: http://build.chromium.org/buildbot/perf/xp-release-dual-core/intl1/report.html?history=100&rev=56174 Only 0.5%. Blame: Possibly 56044 (WebKit merge 65180:65318) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=56176

Patch Set 1 #

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

Messages

Total messages: 3 (0 generated)
nsylvain
10 years, 4 months ago (2010-08-16 01:20:55 UTC) #1
nsylvain
I'll commit TBR. Ananta: can you look at the perf regression please? It seemed to ...
10 years, 4 months ago (2010-08-16 01:21:58 UTC) #2
chase
10 years, 4 months ago (2010-08-16 20:09:19 UTC) #3
lgtm.  Ananta, will you look into this?  Maybe filing a bug would be good, too.

Powered by Google App Engine
This is Rietveld 408576698