Android: allows using adb from a chromium checkout without envsetup.sh

build/android/pylib/constants.py

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Defines a set of constants shared by test runners and other scripts."""
 
 import os
+import subprocess
+import sys
 
 
 CHROME_DIR = os.path.abspath(os.path.join(os.path.dirname(__file__),
                                           os.pardir, os.pardir, os.pardir))
 EMULATOR_SDK_ROOT = os.path.abspath(os.path.join(CHROME_DIR, os.pardir,
                                                  os.pardir))
 
 CHROME_PACKAGE = 'com.google.android.apps.chrome'
 CHROME_ACTIVITY = 'com.google.android.apps.chrome.Main'
 CHROME_DEVTOOLS_SOCKET = 'chrome_devtools_remote'
@@ skipping 52 matching lines @@
 # The directory on the device where perf test output gets saved to.
 DEVICE_PERF_OUTPUT_DIR = '/data/data/' + CHROME_PACKAGE + '/files'
 
 SCREENSHOTS_DIR = os.path.join(CHROME_DIR, 'out_screenshots')
 
 ANDROID_SDK_VERSION = 17
 ANDROID_SDK_ROOT = os.path.join(CHROME_DIR, 'third_party/android_tools/sdk')
 ANDROID_NDK_ROOT = os.path.join(CHROME_DIR, 'third_party/android_tools/ndk')
 
 UPSTREAM_FLAKINESS_SERVER = 'test-results.appspot.com'
+
+
+_FULLY_QUALIFIED_ADB = None
+
+
+def GetADBPath():

[frankf, 2013/06/04 19:50:47]

modules are imported once in python. Can't you just run this inline and get
constants.FULLY_QUILIFIED_ADB directly?

[bulach, 2013/06/05 08:18:27]

good idea! inverted here, so the function is internal and the constant is the
only thing exposed.

+  global _FULLY_QUALIFIED_ADB
+  if _FULLY_QUALIFIED_ADB:
+    return _FULLY_QUALIFIED_ADB
+  if os.environ.get('ANDROID_SDK_ROOT'):
+    _FULLY_QUALIFIED_ADB = 'adb'
+    return _FULLY_QUALIFIED_ADB
+  # If envsetup.sh hasn't been sourced and there's no adb in the path,
+  # set it here.
+  try:
+    with file(os.devnull, 'w') as devnull:
+      subprocess.call(['adb', 'version'], stdout=devnull, stderr=devnull)
+    _FULLY_QUALIFIED_ADB = 'adb'
+  except OSError:
+    print 'No adb found in $PATH, fallback to checked in binary.'
+    _FULLY_QUALIFIED_ADB = os.path.join(
+      CHROME_DIR, 'third_party', 'android_tools', 'sdk', 'platform-tools',
+      'adb')
+  return _FULLY_QUALIFIED_ADB