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

Issue 8916017: updated expecatations (Closed)

Created:
9 years ago by ramant (doing other things)
Modified:
9 years ago
Reviewers:
cmp, Nico
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Patch Set 1 #

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

Messages

Total messages: 4 (0 generated)
ramant (doing other things)
9 years ago (2011-12-14 22:32:06 UTC) #1
Nico
Hi ramant, chrome-si regressions are not acceptable at this point. Instead of adjusting the expectations, ...
9 years ago (2011-12-19 18:34:16 UTC) #2
ramant (doing other things)
On 2011/12/19 18:34:16, Nico wrote: > Hi ramant, > > chrome-si regressions are not acceptable ...
9 years ago (2011-12-19 18:40:41 UTC) #3
cmp
9 years ago (2011-12-19 19:07:49 UTC) #4
lgtm for the expectations change

What Nico says about regressions is generally true.  However, we often have to
cover the unexpected regression state with an expected value to keep the bot
from staying red and masking other errors.  Nico, I'll assume that discussion
will continue in bug 107594.

Powered by Google App Engine
This is Rietveld 408576698