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

Unified Diff: chrome/browser/ui/browser_browsertest.cc

Issue 1828203005: Expose SPDY pushes in DevTools (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: updated wording on push_end comment Created 4 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: chrome/browser/ui/browser_browsertest.cc
diff --git a/chrome/browser/ui/browser_browsertest.cc b/chrome/browser/ui/browser_browsertest.cc
index 1308c25be5e56207e516c15962e6602ab9895f8d..9ee238fecac4368efb0714ae5c9bd03f17e27f6c 100644
--- a/chrome/browser/ui/browser_browsertest.cc
+++ b/chrome/browser/ui/browser_browsertest.cc
@@ -3094,27 +3094,6 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, SecurityStyleChangedObserverGoBack) {
namespace {
-// A URLRequestMockHTTPJob that mocks an SSL connection with an
-// obsolete protocol version.
-class URLRequestNonsecureConnection : public net::URLRequestMockHTTPJob {
- public:
- void GetResponseInfo(net::HttpResponseInfo* info) override {
- info->ssl_info.connection_status = (net::SSL_CONNECTION_VERSION_TLS1_1
- << net::SSL_CONNECTION_VERSION_SHIFT);
- // TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 from
- // http://www.iana.org/assignments/tls-parameters/tls-parameters.xml#tls-parameters-4
- const uint16_t ciphersuite = 0xc02f;
- net::SSLConnectionStatusSetCipherSuite(ciphersuite,
- &info->ssl_info.connection_status);
- }
-
- protected:
- ~URLRequestNonsecureConnection() override {}
-
- private:
- DISALLOW_COPY_AND_ASSIGN(URLRequestNonsecureConnection);
-};
-
class BrowserTestNonsecureURLRequest : public BrowserTest {
public:
BrowserTestNonsecureURLRequest() : BrowserTest() {}
@@ -3124,7 +3103,7 @@ class BrowserTestNonsecureURLRequest : public BrowserTest {
content::BrowserThread::PostTask(
content::BrowserThread::IO, FROM_HERE,
base::Bind(
- &URLRequestNonsecureConnection::AddUrlHandlers, root_http,
+ &net::URLRequestMockHTTPJob::AddUrlHandlers, root_http,
make_scoped_refptr(content::BrowserThread::GetBlockingPool())));
}
@@ -3143,7 +3122,7 @@ IN_PROC_BROWSER_TEST_F(BrowserTestNonsecureURLRequest,
SecurityStyleTestObserver observer(web_contents);
ui_test_utils::NavigateToURL(
- browser(), URLRequestNonsecureConnection::GetMockHttpsUrl(std::string()));
+ browser(), net::URLRequestMockHTTPJob::GetMockHttpsUrl(std::string()));
for (const auto& explanation :
observer.latest_explanations().secure_explanations) {
EXPECT_NE(l10n_util::GetStringUTF8(IDS_SECURE_PROTOCOL_AND_CIPHERSUITE),

Powered by Google App Engine
This is Rietveld 408576698