[Android] Implement gtest and local in platform mode.

build/android/pylib/gtest/gtest_test_instance.py

 # Copyright 2014 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.
 
 import logging
 import os
 import shutil
 import sys
 
 from pylib import constants
@@ skipping 20 matching lines @@
     'chrome/test/data/safari_import',
     'chrome/test/data/scroll',
     'chrome/test/data/third_party',
     'third_party/hunspell_dictionaries/*.dic',
     # crbug.com/258690
     'webkit/data/bmp_decoder',
     'webkit/data/ico_decoder',
 ]
 
 
+# TODO(jbudorick): Make this a classmethod of GtestTestInstance once

[klundberg, 2014/12/09 02:30:47]

s/classmethod/class method 

[jbudorick, 2014/12/09 15:46:51]

Done.

+# test_package_apk and test_package_exe are gone.
+def ParseGTestListTests(raw_list):
+  """Parses a raw test list as provided by --gtest_list_tests.
+
+  Args:
+    raw_list: The raw test listing with the following format:
+
+    IPCChannelTest.
+      SendMessageInChannelConnected
+    IPCSyncChannelTest.
+      Simple
+      DISABLED_SendWithTimeoutMixedOKAndTimeout
+
+  Returns:
+    A list of all tests. For the above raw listing:
+
+    [IPCChannelTest.SendMessageInChannelConnected, IPCSyncChannelTest.Simple,
+     IPCSyncChannelTest.DISABLED_SendWithTimeoutMixedOKAndTimeout]
+  """
+  ret = []
+  current = ''
+  for test in raw_list:
+    if not test:
+      continue
+    if test[0] != ' ':
+      test_case = test.split()[0]
+      if test_case.endswith('.'):
+        current = test_case
+    elif not 'YOU HAVE' in test:
+      test_name = test.split()[0]
+      ret += [current + test_name]
+  return ret
+
+
 class GtestTestInstance(test_instance.TestInstance):
 
-  def __init__(self, options, isolate_delegate):
+  def __init__(self, args, isolate_delegate):
     super(GtestTestInstance, self).__init__()
+    # TODO(jbudorick): accept theirs during merge conflicts

[klundberg, 2014/12/09 02:30:47]

I'm assuming this is just a note for yourself until you have merged with Randy's
CL?

[jbudorick, 2014/12/09 15:46:51]

Yes, it is. I will probably hold this CL until Randy's lands, as I think his is
more urgent.

     self._apk_path = os.path.join(
-        constants.GetOutDirectory(), '%s_apk' % options.suite_name,
-        '%s-debug.apk' % options.suite_name)
+        constants.GetOutDirectory(), '%s_apk' % args.suite_name[0],
+        '%s-debug.apk' % args.suite_name[0])
     self._data_deps = []
-    self._gtest_filter = options.test_filter
-    if options.isolate_file_path:
-      self._isolate_abs_path = os.path.abspath(options.isolate_file_path)
+    self._gtest_filter = args.test_filter
+    if args.isolate_file_path:
+      self._isolate_abs_path = os.path.abspath(args.isolate_file_path)
       self._isolate_delegate = isolate_delegate
       self._isolated_abs_path = os.path.join(
-          constants.GetOutDirectory(), '%s.isolated' % options.suite_name)
+          constants.GetOutDirectory(), '%s.isolated' % args.suite_name)
     else:
       logging.warning('No isolate file provided. No data deps will be pushed.');
       self._isolate_delegate = None
-    self._suite = options.suite_name
+    self._suite = args.suite_name
 
   #override
   def TestType(self):
     return 'gtest'
 
   #override
   def SetUp(self):
     """Map data dependencies via isolate."""
     if self._isolate_delegate:
       self._isolate_delegate.Remap(
@@ skipping 59 matching lines @@
       self._isolate_delegate.Clear()
 
   @property
   def apk(self):
     return self._apk_path
 
   @property
   def suite(self):
     return self._suite