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

Unified Diff: net/http/http_auth_gssapi_posix_unittest.cc

Issue 2826273004: Enable loading gssapi library for Chromad. (Closed)
Patch Set: Enable loading gssapi library for Chromad. Created 3 years, 8 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_gssapi_posix_unittest.cc
diff --git a/net/http/http_auth_gssapi_posix_unittest.cc b/net/http/http_auth_gssapi_posix_unittest.cc
index 21eecacf35b07fd95bf7ac4ace3c1d0759f9aa35..61b16de0bc3c31ed7fcb13a39fbeee45d450b7b4 100644
--- a/net/http/http_auth_gssapi_posix_unittest.cc
+++ b/net/http/http_auth_gssapi_posix_unittest.cc
@@ -83,15 +83,25 @@ TEST(HttpAuthGSSAPIPOSIXTest, GSSAPIStartup) {
// TODO(ahendrickson): Manipulate the libraries and paths to test each of the
// libraries we expect, and also whether or not they have the interface
// functions we want.
- std::unique_ptr<GSSAPILibrary> gssapi(new GSSAPISharedLibrary(std::string()));
+ std::unique_ptr<GSSAPILibrary> gssapi(
+ new GSSAPISharedLibrary(std::string(), true));
DCHECK(gssapi.get());
EXPECT_TRUE(gssapi.get()->Init());
}
+#if defined(OS_CHROMEOS)
+TEST(HttpAuthGSSAPIPOSIXTest, BlockGssapiLibraryLoad) {
+ std::unique_ptr<GSSAPILibrary> gssapi(
+ new GSSAPISharedLibrary(std::string(), false));
+ DCHECK(gssapi.get());
+ EXPECT_FALSE(gssapi.get()->Init());
+}
+#endif
+
#if defined(DLOPEN_KERBEROS)
TEST(HttpAuthGSSAPIPOSIXTest, GSSAPILoadCustomLibrary) {
std::unique_ptr<GSSAPILibrary> gssapi(
- new GSSAPISharedLibrary("/this/library/does/not/exist"));
+ new GSSAPISharedLibrary("/this/library/does/not/exist", true));
EXPECT_FALSE(gssapi.get()->Init());
}
#endif // defined(DLOPEN_KERBEROS)

Powered by Google App Engine
This is Rietveld 408576698