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

Issue 12025025: [Android] Track individual test case timeouts for gtests. (Closed)

Created:
7 years, 11 months ago by frankf
Modified:
7 years, 11 months ago
Reviewers:
craigdh, Isaac (away)
CC:
chromium-reviews, klundberg+watch_chromium.org, frankf+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy+watch_chromium.org
Visibility:
Public.

Description

[Android] Track individual test case timeouts for gtests. BUG=170472 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=177761

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+39 lines, -17 lines) Patch
M build/android/pylib/base/test_result.py View 8 chunks +29 lines, -13 lines 0 comments Download
M build/android/pylib/gtest/test_package.py View 4 chunks +10 lines, -4 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
frankf
TestResults module is crap and will be redesigned soon. Please focus on functionality for this ...
7 years, 11 months ago (2013-01-18 19:56:07 UTC) #1
Isaac (away)
Agreed, structure of python code is not ideal... But this lgtm. Thank you for the ...
7 years, 11 months ago (2013-01-18 20:00:48 UTC) #2
craigdh
7 years, 11 months ago (2013-01-18 21:23:32 UTC) #3
functionality lgtm.

Powered by Google App Engine
This is Rietveld 408576698