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

Issue 11830011: [Telemetry] Put adb on the path if it is not already there. (Closed)

Created:
7 years, 11 months ago by tonyg
Modified:
7 years, 11 months ago
Reviewers:
nduca
CC:
chromium-reviews, chrome-speed-team+watch_google.com, pam+watch_chromium.org, telemetry+watch_chromium.org, eseidel, wiltzius
Visibility:
Public.

Description

[Telemetry] Put adb on the path if it is not already there. There is a linux adb binary checked in to platform-tools. We use it if on linux and the adb command isn't found. BUG=None TEST=tools/perf/run_multipage_benchmarks --browser=list -v Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175659

Patch Set 1 #

Patch Set 2 : Use absolute path #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -3 lines) Patch
M tools/telemetry/telemetry/android_browser_finder.py View 1 2 chunks +10 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
tonyg
ptal
7 years, 11 months ago (2013-01-09 01:34:11 UTC) #1
nduca
lgtm
7 years, 11 months ago (2013-01-09 01:36:52 UTC) #2
commit-bot: I haz the power
7 years, 11 months ago (2013-01-09 01:42:00 UTC) #3

Powered by Google App Engine
This is Rietveld 408576698