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

Issue 465723002: Run android tests through runtest.py. (Closed)

Created:
6 years, 4 months ago by zty
Modified:
6 years, 4 months ago
CC:
chromium-reviews, klundberg+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Run android tests through runtest.py. This CL re-lands https://codereview.chromium.org/414383002. It fixed a bug where debug testers failed to find binaries. TryServer failed to catch this bug due to the fact that rel and dbg testers are sharing the same checkout on bots. https://codereview.chromium.org/452263002/ was submitted to make sure any tester that downloads and extracts archive will clean it up. BUG=329102 R=yfriedman@chromium.org, navabi@chromium.org, jbudorick@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=289231

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+28 lines, -2 lines) Patch
M build/android/buildbot/bb_device_steps.py View 2 chunks +28 lines, -2 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
zty
I sincerely hope this is the last time I bug you guys about this one. ...
6 years, 4 months ago (2014-08-11 22:21:22 UTC) #1
navabi
lgtm. thanks for fixing the problem that caused the previous CL to mistakenly pass the ...
6 years, 4 months ago (2014-08-12 01:30:58 UTC) #2
navabi1
lgtm
6 years, 4 months ago (2014-08-12 21:37:27 UTC) #3
zty
The CQ bit was checked by zty@chromium.org
6 years, 4 months ago (2014-08-12 21:39:06 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/zty@chromium.org/465723002/1
6 years, 4 months ago (2014-08-12 21:45:30 UTC) #5
zty
The CQ bit was unchecked by zty@chromium.org
6 years, 4 months ago (2014-08-12 21:52:24 UTC) #6
zty
The CQ bit was checked by zty@chromium.org
6 years, 4 months ago (2014-08-13 03:12:44 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/zty@chromium.org/465723002/1
6 years, 4 months ago (2014-08-13 03:16:20 UTC) #8
Paweł Hajdan Jr.
LGTM
6 years, 4 months ago (2014-08-13 09:31:58 UTC) #9
commit-bot: I haz the power
6 years, 4 months ago (2014-08-13 09:36:42 UTC) #10
Message was sent while issue was closed.
Change committed as 289231

Powered by Google App Engine
This is Rietveld 408576698