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

Issue 54423002: [Sheriff] Mark flaky test. (Closed)

Created:
7 years, 1 month ago by Michael Achenbach
Modified:
7 years, 1 month ago
Reviewers:
Jakob Kummerow
CC:
v8-dev
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1

Patch Set 2 : Addressed review comments. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -1 line) Patch
M test/mjsunit/mjsunit.status View 1 1 chunk +7 lines, -0 lines 0 comments Download
M tools/testrunner/local/statusfile.py View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Michael Achenbach
PTAL
7 years, 1 month ago (2013-10-31 09:54:23 UTC) #1
Jakob Kummerow
LGTM with comment. https://codereview.chromium.org/54423002/diff/1/test/mjsunit/mjsunit.status File test/mjsunit/mjsunit.status (right): https://codereview.chromium.org/54423002/diff/1/test/mjsunit/mjsunit.status#newcode41 test/mjsunit/mjsunit.status:41: 'arch == nacl_x64'), PASS, FLAKY]], no ...
7 years, 1 month ago (2013-10-31 10:33:44 UTC) #2
Michael Achenbach
7 years, 1 month ago (2013-10-31 11:48:38 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 manually as r17444.

Powered by Google App Engine
This is Rietveld 408576698