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

Issue 12995009: Parse the test executable output on the fly. (Closed)

Created:
7 years, 9 months ago by M-A Ruel
Modified:
7 years, 9 months ago
Reviewers:
csharp
CC:
chromium-reviews
Visibility:
Public.

Description

Parse the test executable output on the fly. This will greatly reduce the latency of output and reduce the likelihood of hangs for long clusters. More granular timeouts will be done in a follow up CL. There isn't much testing since this CL changes the latency of the output but the actual output itself is not changed; there's no "API" change, simply more eager parsing. R=csharp@chromium.org BUG= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=189851

Patch Set 1 #

Patch Set 2 : oops #

Unified diffs Side-by-side diffs Delta from patch set Stats (+77 lines, -21 lines) Patch
M run_test_cases.py View 1 4 chunks +45 lines, -21 lines 0 comments Download
M tests/run_test_cases_test.py View 1 chunk +32 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
M-A Ruel
7 years, 9 months ago (2013-03-22 00:21:01 UTC) #1
csharp
LGTM wow, generators all the way down
7 years, 9 months ago (2013-03-22 15:14:21 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/12995009/3001
7 years, 9 months ago (2013-03-22 18:03:25 UTC) #3
commit-bot: I haz the power
7 years, 9 months ago (2013-03-22 18:04:05 UTC) #4
Message was sent while issue was closed.
Change committed as 189851

Powered by Google App Engine
This is Rietveld 408576698