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

Issue 7046110: Make TrialDirMixIn so it can be used stand alone. (Closed)

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

Description

Make TrialDirMixIn so it can be used stand alone. This will be used in commit queue tests. R=dpranke@chromium.org BUG= TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=88986

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -2 lines) Patch
M tests/trial_dir.py View 2 chunks +13 lines, -2 lines 1 comment Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
9 years, 6 months ago (2011-06-13 15:46:42 UTC) #1
Dirk Pranke
9 years, 6 months ago (2011-06-14 06:22:09 UTC) #2
LGTM.

http://codereview.chromium.org/7046110/diff/1/tests/trial_dir.py
File tests/trial_dir.py (right):

http://codereview.chromium.org/7046110/diff/1/tests/trial_dir.py#newcode69
tests/trial_dir.py:69: """Stand alone mixin."""
This comment doesn't tell me anything about what this is, and worse, I think
these three words are self-contradictory. I'd probably just delete it.

Powered by Google App Engine
This is Rietveld 408576698