[Android] Fix gtest test scripts for non-default CHROMIUM_OUT_DIR.

build/android/pylib/gtest/setup.py

Index: build/android/pylib/gtest/setup.py
diff --git a/build/android/pylib/gtest/setup.py b/build/android/pylib/gtest/setup.py
index 3f2340f88a22bdee8eb73b4a1355b42091218d82..5fe0bedb6b5ef10a1d2674517dafdbc4c0ae6dc8 100644
--- a/build/android/pylib/gtest/setup.py
+++ b/build/android/pylib/gtest/setup.py
@@ -162,19 +162,20 @@ def _GenerateDepsDirUsingIsolate(suite_name):
   # On Android, all pak files need to be in the top-level 'paks' directory.
   paks_dir = os.path.join(constants.ISOLATE_DEPS_DIR, 'paks')
   os.mkdir(paks_dir)
-  for root, _, filenames in os.walk(os.path.join(constants.ISOLATE_DEPS_DIR,
-                                                 'out')):
+
+  deps_out_dir = os.path.join(
+      constants.ISOLATE_DEPS_DIR, os.environ.get('CHROMIUM_OUT_DIR', 'out'))

[frankf, 2014/03/05 01:33:59]

You're duplicating constants.GetOutDirectory(). Also what happened to patch to
get rid of this env var?

[jbudorick, 2014/03/07 07:15:28]

I tried constants.GetOutDirectory() at first because I thought the same thing.
Unfortunately, that appends the build type to the end of CHROMIUM_OUT_DIR --
e.g., it would return 'out/Debug', while this has to be 'out'.

I don't think I ever posted a patch to get rid of this environment variable,
though I did play around with the idea locally. You suggested it in
https://codereview.chromium.org/144093008

[frankf, 2014/03/10 17:35:34]

Why can't we do os.path.join(constants.GetOutDirectory(), os.pardir)

On 2014/03/07 07:15:28, jbudorick wrote:

[jbudorick, 2014/03/11 14:25:09]

We can, and I'm ok with doing so, but how is that any better?

[frankf, 2014/03/11 16:27:34]

It's DRY. Also you're encapsulating how the path is obtained.

On 2014/03/11 14:25:09, jbudorick wrote:

+  for root, _, filenames in os.walk(deps_out_dir):
     for filename in fnmatch.filter(filenames, '*.pak'):
       shutil.move(os.path.join(root, filename), paks_dir)
 
   # Move everything in PRODUCT_DIR to top level.
-  deps_product_dir = os.path.join(constants.ISOLATE_DEPS_DIR, 'out',
-      constants.GetBuildType())
+  deps_product_dir = os.path.join(deps_out_dir, constants.GetBuildType())
   if os.path.isdir(deps_product_dir):
     for p in os.listdir(deps_product_dir):
       shutil.move(os.path.join(deps_product_dir, p), constants.ISOLATE_DEPS_DIR)
     os.rmdir(deps_product_dir)
-    os.rmdir(os.path.join(constants.ISOLATE_DEPS_DIR, 'out'))
+    os.rmdir(deps_out_dir)
 
 
 def _GetDisabledTestsFilterFromFile(suite_name):