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

Issue 10541064: Add Strace parallel support. (Closed)

Created:
8 years, 6 months ago by M-A Ruel
Modified:
8 years, 6 months ago
Reviewers:
cmp
CC:
chromium-reviews, pam+watch_chromium.org
Visibility:
Public.

Description

Add Strace parallel support. Do not add it to Dtrace and LogmanTrace yet since it requires more surgery. So only enable the smoke test on linux. R=cmp@chromium.org BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=141264

Patch Set 1 #

Patch Set 2 : Remove maxDiff #

Unified diffs Side-by-side diffs Delta from patch set Stats (+210 lines, -39 lines) Patch
M tools/isolate/trace_inputs.py View 16 chunks +131 lines, -39 lines 0 comments Download
M tools/isolate/trace_inputs_smoke_test.py View 1 2 chunks +79 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
M-A Ruel
8 years, 6 months ago (2012-06-07 22:39:25 UTC) #1
cmp
lgtm
8 years, 6 months ago (2012-06-08 00:27:07 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/10541064/1
8 years, 6 months ago (2012-06-08 18:36:38 UTC) #3
commit-bot: I haz the power
Presubmit check for 10541064-1 failed and returned exit status 1. Running presubmit commit checks ...
8 years, 6 months ago (2012-06-08 18:37:13 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/maruel@chromium.org/10541064/5001
8 years, 6 months ago (2012-06-08 18:39:50 UTC) #5
commit-bot: I haz the power
8 years, 6 months ago (2012-06-08 19:55:30 UTC) #6
Change committed as 141264

Powered by Google App Engine
This is Rietveld 408576698