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

Issue 2366213005: V8: Reduce test expectations by whitelisting

Created:
4 years, 2 months ago by Michael Achenbach
Modified:
4 years, 1 month ago
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

V8: Reduce test expectations by whitelisting BUG=

Patch Set 1 #

Patch Set 2 : also end result #

Patch Set 3 : New post process #

Total comments: 2

Patch Set 4 : updates #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -3682 lines) Patch
M scripts/slave/recipes/v8.py View 1 2 3 2 chunks +4 lines, -1 line 1 comment Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux_swarming_collect_failure.json View 1 2 3 4 chunks +7 lines, -3681 lines 0 comments Download

Messages

Total messages: 10 (6 generated)
iannucci1
https://codereview.chromium.org/2366213005/diff/40001/scripts/slave/recipes/v8.py File scripts/slave/recipes/v8.py (right): https://codereview.chromium.org/2366213005/diff/40001/scripts/slave/recipes/v8.py#newcode129 scripts/slave/recipes/v8.py:129: api.post_process(Filter('$result')) Hm... I would expect this to remove all ...
4 years, 1 month ago (2016-11-01 20:21:51 UTC) #6
Michael Achenbach
https://codereview.chromium.org/2366213005/diff/40001/scripts/slave/recipes/v8.py File scripts/slave/recipes/v8.py (right): https://codereview.chromium.org/2366213005/diff/40001/scripts/slave/recipes/v8.py#newcode129 scripts/slave/recipes/v8.py:129: api.post_process(Filter('$result')) On 2016/11/01 20:21:51, iannucci1 wrote: > Hm... I ...
4 years, 1 month ago (2016-11-01 20:25:20 UTC) #7
Michael Achenbach
Nice, looks as expected now. I'll add a few more filters before landing.
4 years, 1 month ago (2016-11-01 20:38:46 UTC) #9
tandrii(chromium)
4 years, 1 month ago (2016-11-02 23:08:58 UTC) #10

Powered by Google App Engine
This is Rietveld 408576698