Android: Avoid multiple data file pushes for unittests

build/android/pylib/gtest/test_runner.py

Index: build/android/pylib/gtest/test_runner.py
diff --git a/build/android/pylib/gtest/test_runner.py b/build/android/pylib/gtest/test_runner.py
index 4cfa820363390faa9acf0c43811065039e65eb08..f6f2168233e9050da732e72d1a412b5fa27e0def 100644
--- a/build/android/pylib/gtest/test_runner.py
+++ b/build/android/pylib/gtest/test_runner.py
@@ -30,7 +30,11 @@ def _GetDataFilesForTestSuite(test_suite_basename):
   # Ideally, we'd just push all test data. However, it has >100MB, and a lot
   # of the files are not relevant (some are used for browser_tests, others for
   # features not supported, etc..).
-  if test_suite_basename in ['base_unittests', 'sql_unittests', 'unit_tests']:

[frankf, 2013/04/08 19:19:37]

sql_unittests don't have any deps?

[nilesh, 2013/04/08 23:50:57]

No, no files are needed for sql_unittests.

+  if test_suite_basename == 'base_unittests':
+    return [
+        'base/test/data/',
+    ]
+  elif test_suite_basename == 'unit_tests':
     test_files = [
         'base/test/data/',
         'chrome/test/data/download-test1.lib',
@@ -56,27 +60,22 @@ def _GetDataFilesForTestSuite(test_suite_basename):
         'chrome/test/data/History/',
         'chrome/test/data/json_schema_validator/',
         'chrome/test/data/pref_service/',
+        'chrome/test/data/simple_open_search.xml',
         'chrome/test/data/top_sites/',
         'chrome/test/data/web_app_info/',
         'chrome/test/data/web_database',
         'chrome/test/data/webui/',
         'chrome/test/data/zip',
         'chrome/third_party/mock4js/',
-        'content/browser/gpu/software_rendering_list.json',
-        'net/data/cache_tests/insert_load1',
-        'net/data/cache_tests/dirty_entry5',
-        'net/data/ssl/certificates/',
+        'net/data/ssl/certificates',

[frankf, 2013/04/08 19:19:37]

I'm assuming trailing slash is optional.

[nilesh, 2013/04/08 23:50:57]

yes.

         'third_party/accessibility-developer-tools/gen/axs_testing.js',
-        'ui/base/test/data/data_pack_unittest',
     ]
-    if test_suite_basename == 'unit_tests':
-      test_files += ['chrome/test/data/simple_open_search.xml']
-      # The following are spell check data. Now only list the data under
-      # third_party/hunspell_dictionaries which are used by unit tests.
-      old_cwd = os.getcwd()
-      os.chdir(constants.CHROME_DIR)
-      test_files += glob.glob('third_party/hunspell_dictionaries/*.bdic')
-      os.chdir(old_cwd)
+    # The following are spell check data. Now only list the data under
+    # third_party/hunspell_dictionaries which are used by unit tests.
+    old_cwd = os.getcwd()
+    os.chdir(constants.CHROME_DIR)
+    test_files += glob.glob('third_party/hunspell_dictionaries/*.bdic')
+    os.chdir(old_cwd)
     return test_files
   elif test_suite_basename == 'media_unittests':
     return [
@@ -91,9 +90,9 @@ def _GetDataFilesForTestSuite(test_suite_basename):
         'net/data/ftp',
         'net/data/proxy_resolver_v8_tracing_unittest',
         'net/data/proxy_resolver_v8_unittest',
+        'net/data/proxy_script_fetcher_unittest',
         'net/data/ssl/certificates',
         'net/data/url_request_unittest/',
-        'net/data/proxy_script_fetcher_unittest'
         ]
   elif test_suite_basename == 'ui_tests':
     return [
@@ -101,12 +100,17 @@ def _GetDataFilesForTestSuite(test_suite_basename):
         'chrome/test/data/json2.js',
         'chrome/test/data/sunspider',
         ]
+  elif test_suite_basename == 'ui_unittests':
+    return [
+        'ui/base/test/data/data_pack_unittest/truncated-header.pak',
+    ]
   elif test_suite_basename == 'content_unittests':
     return [
+        'content/browser/gpu/software_rendering_list.json',
         'content/test/data/gpu/webgl_conformance_test_expectations.txt',
         'net/data/ssl/certificates/',
-        'webkit/data/dom_storage/webcore_test_database.localstorage',
         'third_party/hyphen/hyph_en_US.dic',
+        'webkit/data/dom_storage/webcore_test_database.localstorage',
         ]
   elif test_suite_basename == 'cc_perftests':
     return [