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

Side by Side Diff: chrome/browser/net/errorpage_browsertest.cc

Issue 2656373002: Revert of Enable HTTP/0.9 on non-standard ports by default (Closed)
Patch Set: Oops Created 3 years, 10 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 unified diff | Download patch
« no previous file with comments | « chrome/browser/io_thread.cc ('k') | net/http/http_network_session.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <memory> 5 #include <memory>
6 #include <utility> 6 #include <utility>
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 1476 matching lines...) Expand 10 before | Expand all | Expand 10 after
1487 // Make sure error page shows correct unicode for IDN. 1487 // Make sure error page shows correct unicode for IDN.
1488 IN_PROC_BROWSER_TEST_F(ErrorPageForIDNTest, IDN) { 1488 IN_PROC_BROWSER_TEST_F(ErrorPageForIDNTest, IDN) {
1489 // ERR_UNSAFE_PORT will not trigger navigation corrections. 1489 // ERR_UNSAFE_PORT will not trigger navigation corrections.
1490 ui_test_utils::NavigateToURL( 1490 ui_test_utils::NavigateToURL(
1491 browser(), 1491 browser(),
1492 URLRequestFailedJob::GetMockHttpUrlForHostname(net::ERR_UNSAFE_PORT, 1492 URLRequestFailedJob::GetMockHttpUrlForHostname(net::ERR_UNSAFE_PORT,
1493 kHostname)); 1493 kHostname));
1494 EXPECT_TRUE(IsDisplayingText(browser(), kHostnameJSUnicode)); 1494 EXPECT_TRUE(IsDisplayingText(browser(), kHostnameJSUnicode));
1495 } 1495 }
1496 1496
1497 // Make sure HTTP/0.9 is enabled on non-default ports by default. 1497 // Make sure HTTP/0.9 is disabled on non-default ports by default.
1498 IN_PROC_BROWSER_TEST_F(ErrorPageTest, Http09WeirdPort) { 1498 IN_PROC_BROWSER_TEST_F(ErrorPageTest, Http09WeirdPort) {
1499 const char kHttp09Response[] = "JumboShrimp";
1500 ASSERT_TRUE(embedded_test_server()->Start()); 1499 ASSERT_TRUE(embedded_test_server()->Start());
1501 ui_test_utils::NavigateToURL( 1500 ui_test_utils::NavigateToURL(
1502 browser(), embedded_test_server()->GetURL(std::string("/echo-raw?") + 1501 browser(), embedded_test_server()->GetURL("/echo-raw?spam"));
1503 kHttp09Response)); 1502 ExpectDisplayingLocalErrorPage(browser(), net::ERR_INVALID_HTTP_RESPONSE);
1504 EXPECT_TRUE(IsDisplayingText(browser(), kHttp09Response));
1505 } 1503 }
1506 1504
1507 class ErrorPageWithHttp09OnNonDefaultPortsTest : public InProcessBrowserTest { 1505 class ErrorPageWithHttp09OnNonDefaultPortsTest : public InProcessBrowserTest {
1508 public: 1506 public:
1509 // InProcessBrowserTest: 1507 // InProcessBrowserTest:
1510 void SetUp() override { 1508 void SetUp() override {
1511 EXPECT_CALL(policy_provider_, IsInitializationComplete(testing::_)) 1509 EXPECT_CALL(policy_provider_, IsInitializationComplete(testing::_))
1512 .WillRepeatedly(testing::Return(true)); 1510 .WillRepeatedly(testing::Return(true));
1513 policy::PolicyMap values; 1511 policy::PolicyMap values;
1514 values.Set(policy::key::kHttp09OnNonDefaultPortsEnabled, 1512 values.Set(policy::key::kHttp09OnNonDefaultPortsEnabled,
(...skipping 16 matching lines...) Expand all
1531 Http09WeirdPortEnabled) { 1529 Http09WeirdPortEnabled) {
1532 const char kHttp09Response[] = "JumboShrimp"; 1530 const char kHttp09Response[] = "JumboShrimp";
1533 ASSERT_TRUE(embedded_test_server()->Start()); 1531 ASSERT_TRUE(embedded_test_server()->Start());
1534 ui_test_utils::NavigateToURL( 1532 ui_test_utils::NavigateToURL(
1535 browser(), embedded_test_server()->GetURL(std::string("/echo-raw?") + 1533 browser(), embedded_test_server()->GetURL(std::string("/echo-raw?") +
1536 kHttp09Response)); 1534 kHttp09Response));
1537 EXPECT_TRUE(IsDisplayingText(browser(), kHttp09Response)); 1535 EXPECT_TRUE(IsDisplayingText(browser(), kHttp09Response));
1538 } 1536 }
1539 1537
1540 } // namespace 1538 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/io_thread.cc ('k') | net/http/http_network_session.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698