[test_runner] support relative paths on argument --shared-prefs-file

build/android/test_runner.py

Index: build/android/test_runner.py
diff --git a/build/android/test_runner.py b/build/android/test_runner.py
index b22607fa4c1b34fc3e94f4bf6fd5755ea12baac1..e2212a814aca0ec0863ee0d5f2be28710cf69ef4 100755
--- a/build/android/test_runner.py
+++ b/build/android/test_runner.py
@@ -50,6 +50,12 @@ _DEVIL_STATIC_CONFIG_FILE = os.path.abspath(os.path.join(
     host_paths.DIR_SOURCE_ROOT, 'build', 'android', 'devil_config.json'))
 
 
+def _RealPath(arg):
+  if arg.startswith('//'):
+    arg = os.path.abspath(os.path.join(host_paths.DIR_SOURCE_ROOT,
+                                       arg[2:].replace('/', os.sep)))

[jbudorick, 2017/07/25 21:30:34]

curious: what's the reasoning behind this replacement?

[shenghuazhang, 2017/07/25 21:38:25]

I was actually not too sure if the os specific separator is necessary here, just
saw a similar situation using the replacement:

https://codesearch.chromium.org/chromium/build/scripts/slave/recipe_modules/c...

[jbudorick, 2017/07/25 21:56:00]

sgtm

+  return os.path.realpath(arg)
+

[jbudorick, 2017/07/25 21:30:34]

nit: +1 blank line

[shenghuazhang, 2017/07/25 21:38:25]

Done.

 def AddTestLauncherOptions(parser):
   """Adds arguments mirroring //base/test/launcher.
 
@@ -420,7 +426,7 @@ def AddInstrumentationTestOptions(parser):
       help='Capture screenshots of test failures')
   parser.add_argument(
       '--shared-prefs-file',
-      dest='shared_prefs_file', type=os.path.realpath,
+      dest='shared_prefs_file', type=_RealPath,
       help='The relative path to a file containing JSON list of shared '
            'preference files to edit and how to do so. Example list: '
            '[{'