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

Unified Diff: components/cronet.gypi

Issue 1448583003: [Cronet] Add QUIC experimental params (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@experiment_ops
Patch Set: Created 5 years, 1 month 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 71d2794ec6eaf7413412921e7663b2586ebde9e5..17e3f46b03fdace9f81ff0ba53b166c502c12aa5 100644
--- a/components/cronet.gypi
+++ b/components/cronet.gypi
@@ -446,6 +446,34 @@
'includes': [ '../build/java_apk.gypi' ],
},
{
+ 'target_name': 'cronet_unittests',
mef 2015/11/18 20:33:25 FWIW cronet_unittests are already included into co
xunjieli 2015/11/18 20:58:44 They are probably executed, but not on Cronet bots
mef 2015/11/18 21:35:33 I don't have particular preference, I just don't t
xunjieli 2015/11/18 21:52:54 Definitely not both. I have moved the other one ov
+ 'type': '<(gtest_target_type)',
+ 'dependencies': [
+ 'cronet_static_small',
+ '../base/base.gyp:base',
+ '../base/base.gyp:test_support_base',
+ '../testing/gtest.gyp:gtest',
+ '../testing/android/native_test.gyp:native_test_native_code',
+ ],
+ 'sources': [
+ 'cronet/run_all_unittests.cc',
+ 'cronet/url_request_context_config_unittest.cc',
+ ],
+ },
+ {
+ 'target_name': 'cronet_unittests_apk',
+ 'type': 'none',
+ 'dependencies': [
+ 'cronet_unittests',
+ ],
+ 'variables': {
+ 'test_suite_name': 'cronet_unittests',
+ },
+ 'includes': [
+ '../build/apk_test.gypi',
+ ],
+ },
+ {
'target_name': 'cronet_package',
'type': 'none',
'dependencies': [

Powered by Google App Engine
This is Rietveld 408576698