Commit 5c10479a authored by jbudorick's avatar jbudorick Committed by Commit bot

[Android] Don't push deps when listing gtests.

PushDataDeps takes a long time even when all files are on the device
because calculating the digests of all the files takes _much_ longer
on a device than it does on a host.

Skipping the push step during test listing -- where the data deps are
unnecessary -- means that we don't try to push files twice in each
gtest run, so we skip all of the device-side MD5 calculations that
would happen in the second run.

BUG=400440

Review URL: https://codereview.chromium.org/560133002

Cr-Commit-Position: refs/heads/master@{#294667}
parent f51d7453
......@@ -213,6 +213,11 @@ def _GetTests(test_options, test_package, devices):
"""
def TestListerRunnerFactory(device, _shard_index):
class TestListerRunner(test_runner.TestRunner):
#override
def PushDataDeps(self):
pass
#override
def RunTest(self, _test):
result = base_test_result.BaseTestResult(
'gtest_list_tests', base_test_result.ResultType.PASS)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment