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

Unified Diff: net/BUILD.gn

Issue 1128043007: Support Kerberos on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix cbentzel@'s nits Created 5 years, 5 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: net/BUILD.gn
diff --git a/net/BUILD.gn b/net/BUILD.gn
index 45931187b29ac8b8e0d448ffdfc712c77dd761ef..7a2b9721f33981ae0ae6c4881dcba1b113099c57 100644
--- a/net/BUILD.gn
+++ b/net/BUILD.gn
@@ -1215,6 +1215,7 @@ if (is_android) {
"android/java/src/org/chromium/net/AndroidNetworkLibrary.java",
"android/java/src/org/chromium/net/AndroidPrivateKey.java",
"android/java/src/org/chromium/net/GURLUtils.java",
+ "android/java/src/org/chromium/net/HttpNegotiateAuthenticator.java",
"android/java/src/org/chromium/net/NetworkChangeNotifier.java",
"android/java/src/org/chromium/net/ProxyChangeListener.java",
"android/java/src/org/chromium/net/X509Util.java",
@@ -1224,6 +1225,7 @@ if (is_android) {
generate_jni("net_test_jni_headers") {
sources = [
"android/javatests/src/org/chromium/net/AndroidKeyStoreTestUtil.java",
+ "test/android/javatests/src/org/chromium/net/test/DummySpnegoAuthenticator.java",
]
jni_package = "net"
}
@@ -1407,14 +1409,20 @@ if (!is_android && !is_mac) {
if (use_kerberos) {
defines += [ "USE_KERBEROS" ]
- } else {
+ }
+
+ # These are excluded on Android, because the actual Kerberos support, which
+ # these test, is in a separate app on Android.
+ if (!use_kerberos || is_android) {
sources -= [
"http/http_auth_gssapi_posix_unittest.cc",
- "http/http_auth_handler_negotiate_unittest.cc",
"http/mock_gssapi_library_posix.cc",
"http/mock_gssapi_library_posix.h",
]
}
+ if (!use_kerberos) {
+ sources -= [ "http/http_auth_handler_negotiate_unittest.cc" ]
+ }
if (use_openssl || (!is_desktop_linux && !is_chromeos && !is_ios)) {
# Only include this test when on Posix and using NSS for

Powered by Google App Engine
This is Rietveld 408576698