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

Unified Diff: net/http/http_auth_unittest.cc

Issue 7633006: Linux: control usage of Kerberos via use_kerberos gyp flag (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: dupe Created 9 years, 4 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/http/http_auth_unittest.cc
diff --git a/net/http/http_auth_unittest.cc b/net/http/http_auth_unittest.cc
index fe11c03e8582d1031ab012c40360e49c8c1df2cc..e70cb0da9ca9fc02abf2daa24b8f75ba26271896 100644
--- a/net/http/http_auth_unittest.cc
+++ b/net/http/http_auth_unittest.cc
@@ -103,14 +103,19 @@ TEST(HttpAuthTest, ChooseBestChallenge) {
"",
},
{
+ "WWW-Authenticate: Negotiate\n"
+ "WWW-Authenticate: NTLM\n",
+
+#if defined(USE_KERBEROS)
// Choose Negotiate over NTLM on all platforms.
// TODO(ahendrickson): This may be flaky on Linux and OSX as it
// relies on being able to load one of the known .so files
// for gssapi.
- "WWW-Authenticate: Negotiate\n"
- "WWW-Authenticate: NTLM\n",
-
HttpAuth::AUTH_SCHEME_NEGOTIATE,
+#else
+ // On systems that don't use Kerberos fall back to NTLM.
+ HttpAuth::AUTH_SCHEME_NTLM,
+#endif // defined(USE_KERBEROS)
"",
}
};

Powered by Google App Engine
This is Rietveld 408576698