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

Issue 10377105: Add scripts to list or trace all test cases in a gtest executable. (Closed)

Created:
8 years, 7 months ago by M-A Ruel
Modified:
8 years, 7 months ago
CC:
chromium-reviews, pam+watch_chromium.org, csharp, MAD
Visibility:
Public.

Description

Add scripts to list or trace all test cases in a gtest executable. list_test_cases.py quickly outputs the test cases in a gtest executable. For browser_tests is takes ~100ms. trace_test_cases.py traces each test individually to be able to generate pretty graphs. read_trace.py is a debugging tool to read a single trace. Refactor trace_inputs.py to make it possible. I changed the semantic for it so that stdin is closed and stderr is redirected to stdout. I also have the logger not look at the logging level to determine if output should be shown. R=rogerta@chromium.org BUG=98636 TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=136601

Patch Set 1 #

Patch Set 2 : . #

Total comments: 2

Patch Set 3 : Add more docstrings #

Patch Set 4 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+545 lines, -136 lines) Patch
M tools/isolate/isolate.py View 1 chunk +2 lines, -2 lines 0 comments Download
M tools/isolate/isolate_smoke_test.py View 2 chunks +2 lines, -2 lines 0 comments Download
A tools/isolate/list_test_cases.py View 1 chunk +102 lines, -0 lines 0 comments Download
A tools/isolate/read_trace.py View 1 2 1 chunk +74 lines, -0 lines 0 comments Download
M tools/isolate/trace_inputs.py View 1 2 12 chunks +156 lines, -132 lines 0 comments Download
A tools/isolate/trace_test_cases.py View 1 1 chunk +209 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
M-A Ruel
8 years, 7 months ago (2012-05-11 13:22:08 UTC) #1
Roger Tawa OOO till Jul 10th
lgtm, some doc nits below. http://codereview.chromium.org/10377105/diff/2001/tools/isolate/read_trace.py File tools/isolate/read_trace.py (right): http://codereview.chromium.org/10377105/diff/2001/tools/isolate/read_trace.py#newcode33 tools/isolate/read_trace.py:33: '-v', '--verbose', action='count', default=0, ...
8 years, 7 months ago (2012-05-11 14:17: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/10377105/1007
8 years, 7 months ago (2012-05-11 15:54:57 UTC) #3
commit-bot: I haz the power
8 years, 7 months ago (2012-05-11 17:25:05 UTC) #4
Change committed as 136601

Powered by Google App Engine
This is Rietveld 408576698