Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(743)

Unified Diff: build/android/single_test_runner.py

Issue 9834037: apk-based test runner work. Not enabled yet. This CL is a combination of upstreaming, ndk/ant-ifi… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: build/android/single_test_runner.py
diff --git a/build/android/single_test_runner.py b/build/android/single_test_runner.py
index ef9971a30a18351571b7eb88a2329799d92823c7..4a93813b47efb97626fe06a87884f230e29f82b4 100644
--- a/build/android/single_test_runner.py
+++ b/build/android/single_test_runner.py
@@ -9,6 +9,7 @@ import sys
from base_test_runner import BaseTestRunner
import debug_info
import run_tests_helper
+from test_package_apk import TestPackageApk
from test_package_executable import TestPackageExecutable
from test_result import TestResults
@@ -46,7 +47,15 @@ class SingleTestRunner(BaseTestRunner):
self.dump_debug_info = None
self.fast_and_loose = fast_and_loose
- self.test_package = TestPackageExecutable(self.adb, device,
+ if os.path.splitext(test_suite)[1] == '.apk':
+ self.test_package = TestPackageApk(self.adb, device,
+ test_suite, timeout, rebaseline, performance_test, cleanup_test_files,
+ tool, self.dump_debug_info)
+ else:
+ android_product_out = '.' # os.environ['ANDROID_PRODUCT_OUT']
+ symbols_dir = os.path.join(android_product_out, 'symbols', 'data',
+ 'local')
bulach 2012/03/23 10:30:48 55-56 seem to be unused?
John Grabowski 2012/03/24 01:59:18 That is true. I left them here to remind me to ma
+ self.test_package = TestPackageExecutable(self.adb, device,
test_suite, timeout, rebaseline, performance_test, cleanup_test_files,
tool, self.dump_debug_info)

Powered by Google App Engine
This is Rietveld 408576698