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

Unified Diff: components/cronet.gypi

Issue 1536433002: [Cronet] Get Cronet performance test running again (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: branch off telemetry change Created 4 years, 11 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: components/cronet.gypi
diff --git a/components/cronet.gypi b/components/cronet.gypi
index b45aeda7c4de0b1afe2c2ae49d3532cdd4435215..f01002c00fa78ee1a832a2718e88f8f7733c2655 100644
--- a/components/cronet.gypi
+++ b/components/cronet.gypi
@@ -352,6 +352,8 @@
'cronet/android/test/network_change_notifier_util.h',
'cronet/android/test/cronet_url_request_context_config_test.cc',
'cronet/android/test/cronet_url_request_context_config_test.h',
+ 'cronet/android/test/cronet_test_util.cc',
+ 'cronet/android/test/cronet_test_util.h',
],
'dependencies': [
'cronet_tests_jni_headers',
@@ -376,15 +378,34 @@
'includes': [ 'cronet/cronet_static.gypi' ],
},
{
+ 'target_name': 'cronet_test_support',
+ 'type': 'none',
+ 'dependencies': [
+ 'cronet_java',
+ '../net/net.gyp:net_java_test_support',
+ ],
+ 'variables': {
+ 'java_in_dir': 'cronet/android/test',
+ 'additional_src_dirs': [ 'cronet/android/test/javatests/src' ],
+ 'run_findbugs': 1,
+ },
+ 'includes': [ '../build/java.gypi' ],
+ },
+ {
'target_name': 'cronet_test_apk',
'type': 'none',
'dependencies': [
'cronet_java',
+ 'cronet_test_support',
'../net/net.gyp:net_java_test_support',
],
'variables': {
'apk_name': 'CronetTest',
+ # There isn't an easy way to have a java_apk target without any Java
+ # so we'll borrow the trick from the net_test_support_apk target of
+ # pointing it at placeholder Java via java_in_dir_suffix.
'java_in_dir': 'cronet/android/test',
+ 'java_in_dir_suffix': '/src_dummy',
'resource_dir': 'cronet/android/test/res',
'asset_location': 'cronet/android/test/assets',
'native_lib_target': 'libcronet_tests',
@@ -427,15 +448,16 @@
'dependencies': [
'cronet_java',
'cronet_api',
+ 'cronet_test_support',
],
'variables': {
'apk_name': 'CronetPerfTest',
'java_in_dir': 'cronet/android/test/javaperftests',
- 'is_test_apk': 1,
- 'native_lib_target': 'libcronet',
+ 'native_lib_target': 'libcronet_tests',
'proguard_enabled': 'true',
'proguard_flags_paths': [
'cronet/android/proguard.cfg',
+ 'cronet/android/test/javaperftests/proguard.cfg',
],
'run_findbugs': 1,
},

Powered by Google App Engine
This is Rietveld 408576698