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

Unified Diff: net/url_request/url_request_unittest.cc

Issue 138263003: Remove Chrome Frame hacks from net tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/url_request/url_request_unittest.cc
diff --git a/net/url_request/url_request_unittest.cc b/net/url_request/url_request_unittest.cc
index 0c058779910db675099b888a33c1b5f1dc8608bf..20c61af15d1cded76306aa3e34bd58daf5bf21ec 100644
--- a/net/url_request/url_request_unittest.cc
+++ b/net/url_request/url_request_unittest.cc
@@ -240,14 +240,6 @@ UploadDataStream* CreateSimpleUploadData(const char* data) {
// Verify that the SSLInfo of a successful SSL connection has valid values.
void CheckSSLInfo(const SSLInfo& ssl_info) {
- // Allow ChromeFrame fake SSLInfo to get through.
- if (ssl_info.cert.get() &&
- ssl_info.cert.get()->issuer().GetDisplayName() == "Chrome Internal") {
- // -1 means unknown.
- EXPECT_EQ(ssl_info.security_bits, -1);
- return;
- }
-
// -1 means unknown. 0 means no encryption.
EXPECT_GT(ssl_info.security_bits, 0);
@@ -2690,10 +2682,7 @@ class URLRequestTestHTTP : public URLRequestTest {
bool is_success = r.status().is_success();
if (!is_success) {
- // Requests handled by ChromeFrame send a less precise error message,
- // ERR_CONNECTION_ABORTED.
- EXPECT_TRUE(r.status().error() == ERR_RESPONSE_HEADERS_TOO_BIG ||
- r.status().error() == ERR_CONNECTION_ABORTED);
+ EXPECT_TRUE(r.status().error() == ERR_RESPONSE_HEADERS_TOO_BIG);
// The test server appears to be unable to handle subsequent requests
// after this error is triggered. Force it to restart.
EXPECT_TRUE(test_server_.Stop());
@@ -5962,10 +5951,7 @@ TEST_F(URLRequestTestHTTP, OverrideUserAgent) {
req.SetExtraRequestHeaders(headers);
req.Start();
base::RunLoop().Run();
- // If the net tests are being run with ChromeFrame then we need to allow for
- // the 'chromeframe' suffix which is added to the user agent before the
- // closing parentheses.
- EXPECT_TRUE(StartsWithASCII(d.data_received(), "Lynx (textmode", true));
+ EXPECT_EQ(std::string("Lynx (textmode)"), d.data_received());
}
// Check that a NULL HttpUserAgentSettings causes the corresponding empty
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698