Commit 61721a49 authored by Andrew Grieve's avatar Andrew Grieve Committed by Commit Bot

Android: Fix generate_gradle.py's handling of _test_apk targets

These targets require building the name of the underlying android_apk
template, which was renamed in 88952e43.

Change-Id: I4cd02165e42a3ff932bcc7c3b605f52248e103fc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1792968
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: default avatarPeter Wen <wnwen@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#694772}
parent 772a281d
...@@ -45,6 +45,7 @@ _CMAKE_FILE = 'CMakeLists.txt' ...@@ -45,6 +45,7 @@ _CMAKE_FILE = 'CMakeLists.txt'
_MODULE_ALL = '_all' _MODULE_ALL = '_all'
_SRC_INTERNAL = os.path.join( _SRC_INTERNAL = os.path.join(
os.path.dirname(host_paths.DIR_SOURCE_ROOT), 'src-internal') os.path.dirname(host_paths.DIR_SOURCE_ROOT), 'src-internal')
_INSTRUMENTATION_TARGET_SUFFIX = '_test_apk__test_apk__apk'
_DEFAULT_TARGETS = [ _DEFAULT_TARGETS = [
'//android_webview/test/embedded_test_server:aw_net_test_support_apk', '//android_webview/test/embedded_test_server:aw_net_test_support_apk',
...@@ -733,8 +734,8 @@ def _CombineTestEntries(entries): ...@@ -733,8 +734,8 @@ def _CombineTestEntries(entries):
android_test_entries = collections.defaultdict(list) android_test_entries = collections.defaultdict(list)
for entry in entries: for entry in entries:
target_name = entry.GnTarget() target_name = entry.GnTarget()
if (target_name.endswith('_test_apk__apk') and if (target_name.endswith(_INSTRUMENTATION_TARGET_SUFFIX)
'apk_under_test' in entry.Gradle()): and 'apk_under_test' in entry.Gradle()):
apk_name = entry.Gradle()['apk_under_test'] apk_name = entry.Gradle()['apk_under_test']
android_test_entries[apk_name].append(entry) android_test_entries[apk_name].append(entry)
else: else:
...@@ -848,8 +849,10 @@ def main(): ...@@ -848,8 +849,10 @@ def main():
targets = _QueryForAllGnTargets(output_dir) targets = _QueryForAllGnTargets(output_dir)
else: else:
assert not args.native_targets, 'Native editing requires --all.' assert not args.native_targets, 'Native editing requires --all.'
targets = [re.sub(r'_test_apk$', '_test_apk__apk', t) targets = [
for t in targets_from_args] re.sub(r'_test_apk$', _INSTRUMENTATION_TARGET_SUFFIX, t)
for t in targets_from_args
]
# Necessary after "gn clean" # Necessary after "gn clean"
if not os.path.exists(os.path.join(output_dir, 'build_vars.txt')): if not os.path.exists(os.path.join(output_dir, 'build_vars.txt')):
_RunGnGen(output_dir) _RunGnGen(output_dir)
...@@ -884,7 +887,7 @@ def main(): ...@@ -884,7 +887,7 @@ def main():
main_entries = [ main_entries = [
e for e in main_entries e for e in main_entries
if (e.GetType() in BASE_TYPES or e.GnTarget() in targets_from_args if (e.GetType() in BASE_TYPES or e.GnTarget() in targets_from_args
or e.GnTarget().endswith('_test_apk__apk')) or e.GnTarget().endswith(_INSTRUMENTATION_TARGET_SUFFIX))
] ]
if args.split_projects: if args.split_projects:
......
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