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

Unified Diff: net/http/http_stream_factory_impl_unittest.cc

Issue 284423002: Remove HttpStreamFactory's NPN/SPDY globals, except for spdy_enabled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix merge Created 6 years, 7 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 | « net/http/http_stream_factory_impl_job.cc ('k') | net/net.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_stream_factory_impl_unittest.cc
diff --git a/net/http/http_stream_factory_impl_unittest.cc b/net/http/http_stream_factory_impl_unittest.cc
index d6daa23dcd60195f722892cc88d3b477911114d9..6e26b2d353a6363b62fc5dab276a1cca7ea2903e 100644
--- a/net/http/http_stream_factory_impl_unittest.cc
+++ b/net/http/http_stream_factory_impl_unittest.cc
@@ -43,20 +43,6 @@ namespace net {
namespace {
-class UseAlternateProtocolsScopedSetter {
- public:
- explicit UseAlternateProtocolsScopedSetter(bool use_alternate_protocols)
- : use_alternate_protocols_(HttpStreamFactory::use_alternate_protocols()) {
- HttpStreamFactory::set_use_alternate_protocols(use_alternate_protocols);
- }
- ~UseAlternateProtocolsScopedSetter() {
- HttpStreamFactory::set_use_alternate_protocols(use_alternate_protocols_);
- }
-
- private:
- bool use_alternate_protocols_;
-};
-
class MockWebSocketHandshakeStream : public WebSocketHandshakeStreamBase {
public:
enum StreamType {
@@ -1265,9 +1251,9 @@ TEST_P(HttpStreamFactoryTest, DISABLED_RequestWebSocketSpdyHandshakeStream) {
// TODO(ricea): Re-enable once WebSocket over SPDY is implemented.
TEST_P(HttpStreamFactoryTest, DISABLED_OrphanedWebSocketStream) {
- UseAlternateProtocolsScopedSetter use_alternate_protocols(true);
SpdySessionDependencies session_deps(GetParam(),
ProxyService::CreateDirect());
+ session_deps.use_alternate_protocols = true;
MockRead mock_read(ASYNC, OK);
DeterministicSocketData socket_data(&mock_read, 1, NULL, 0);
« no previous file with comments | « net/http/http_stream_factory_impl_job.cc ('k') | net/net.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698