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

Unified Diff: net/http/http_stream_factory_impl_unittest.cc

Issue 197283012: Retry requests on reused sockets that receive ERR_EMPTY_RESPONSE. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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: 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 f2185a9512833cf8a2af8d295d18059f52b59583..2813bbebc524a8b1e326743c634b1b032b917600 100644
--- a/net/http/http_stream_factory_impl_unittest.cc
+++ b/net/http/http_stream_factory_impl_unittest.cc
@@ -21,6 +21,7 @@
#include "net/http/http_server_properties.h"
#include "net/http/http_server_properties_impl.h"
#include "net/http/http_stream.h"
+#include "net/http/http_stream_factory_test_util.h"
#include "net/http/transport_security_state.h"
#include "net/proxy/proxy_info.h"
#include "net/proxy/proxy_service.h"
@@ -125,36 +126,6 @@ class MockWebSocketHandshakeStream : public WebSocketHandshakeStreamBase {
const StreamType type_;
};
-// HttpStreamFactoryImpl subclass that can wait until a preconnect is complete.
-class MockHttpStreamFactoryImplForPreconnect : public HttpStreamFactoryImpl {
- public:
- MockHttpStreamFactoryImplForPreconnect(HttpNetworkSession* session,
- bool for_websockets)
- : HttpStreamFactoryImpl(session, for_websockets),
- preconnect_done_(false),
- waiting_for_preconnect_(false) {}
-
-
- void WaitForPreconnects() {
- while (!preconnect_done_) {
- waiting_for_preconnect_ = true;
- base::MessageLoop::current()->Run();
- waiting_for_preconnect_ = false;
- }
- }
-
- private:
- // HttpStreamFactoryImpl methods.
- virtual void OnPreconnectsCompleteInternal() OVERRIDE {
- preconnect_done_ = true;
- if (waiting_for_preconnect_)
- base::MessageLoop::current()->Quit();
- }
-
- bool preconnect_done_;
- bool waiting_for_preconnect_;
-};
-
class StreamRequestWaiter : public HttpStreamRequest::Delegate {
public:
StreamRequestWaiter()

Powered by Google App Engine
This is Rietveld 408576698