From e61ee7d331b036aadd607349e3018955746b3125 Mon Sep 17 00:00:00 2001 From: "agl@chromium.org" Date: Tue, 6 Dec 2011 18:07:19 +0000 Subject: Revert "net: reenable Twitter's public key pins with api.twitter.com using the CDN pins." This reverts commit r113203. It failed on the ChromeOS valgrind and heapcheck bots. BUG=113203 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113229 0039d316-1c4b-4281-b951-d872f2087c98 --- net/base/transport_security_state_unittest.cc | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) (limited to 'net/base/transport_security_state_unittest.cc') diff --git a/net/base/transport_security_state_unittest.cc b/net/base/transport_security_state_unittest.cc index 959805e..b5f3c4d 100644 --- a/net/base/transport_security_state_unittest.cc +++ b/net/base/transport_security_state_unittest.cc @@ -580,10 +580,18 @@ TEST_F(TransportSecurityStateTest, Preloaded) { EXPECT_TRUE(ShouldRedirect("ubertt.org")); EXPECT_TRUE(ShouldRedirect("foo.ubertt.org")); - - EXPECT_FALSE(ShouldRedirect("twitter.com")); - EXPECT_FALSE(ShouldRedirect("www.twitter.com")); - EXPECT_TRUE(HasPins("www.twitter.com")); +#if 0 + // Currently disabled to debug Twitter public key pins --agl +#if defined(OS_CHROMEOS) + EXPECT_TRUE(state.GetDomainState(&domain_state, + "twitter.com", + false)); +#else + EXPECT_FALSE(state.GetDomainState(&domain_state, + "twitter.com", + false)); +#endif +#endif } TEST_F(TransportSecurityStateTest, LongNames) { @@ -684,6 +692,7 @@ TEST_F(TransportSecurityStateTest, BuiltinCertPins) { true)); EXPECT_TRUE(state.HasPinsForHost(&domain_state, "www.googleplex.com", true)); +#if 0 // Disabled in order to help track down pinning failures --agl EXPECT_TRUE(state.HasPinsForHost(&domain_state, "twitter.com", true)); EXPECT_FALSE(state.HasPinsForHost(&domain_state, "foo.twitter.com", true)); @@ -696,6 +705,7 @@ TEST_F(TransportSecurityStateTest, BuiltinCertPins) { EXPECT_TRUE(state.HasPinsForHost(&domain_state, "platform.twitter.com", true)); EXPECT_TRUE(state.HasPinsForHost(&domain_state, "si0.twimg.com", true)); EXPECT_TRUE(state.HasPinsForHost(&domain_state, "twimg0-a.akamaihd.net", true)); +#endif } static bool AddHash(const std::string& type_and_base64, -- cgit v1.1