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

Unified Diff: net/http/http_pipelined_host_impl.h

Issue 8586015: Slow start pipelining. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Patch for landing Created 9 years, 1 month 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_pipelined_host.cc ('k') | net/http/http_pipelined_host_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_pipelined_host_impl.h
diff --git a/net/http/http_pipelined_host_impl.h b/net/http/http_pipelined_host_impl.h
new file mode 100644
index 0000000000000000000000000000000000000000..af78005cfb689c0b8b97aed4f735f33a652b3eb5
--- /dev/null
+++ b/net/http/http_pipelined_host_impl.h
@@ -0,0 +1,99 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef NET_HTTP_HTTP_PIPELINED_HOST_IMPL_H_
+#define NET_HTTP_HTTP_PIPELINED_HOST_IMPL_H_
+#pragma once
+
+#include <map>
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
+#include "net/base/host_port_pair.h"
+#include "net/base/net_export.h"
+#include "net/http/http_pipelined_connection.h"
+#include "net/http/http_pipelined_host.h"
+
+namespace net {
+
+class BoundNetLog;
+class ClientSocketHandle;
+class HttpPipelinedStream;
+class ProxyInfo;
+struct SSLConfig;
+
+// Manages all of the pipelining state for specific host with active pipelined
+// HTTP requests. Manages connection jobs, constructs pipelined streams, and
+// assigns requests to the least loaded pipelined connection.
+class NET_EXPORT_PRIVATE HttpPipelinedHostImpl
+ : public HttpPipelinedHost,
+ public HttpPipelinedConnection::Delegate {
+ public:
+ HttpPipelinedHostImpl(HttpPipelinedHost::Delegate* delegate,
+ const HostPortPair& origin,
+ HttpPipelinedConnection::Factory* factory,
+ Capability capability);
+ virtual ~HttpPipelinedHostImpl();
+
+ // HttpPipelinedHost interface
+ virtual HttpPipelinedStream* CreateStreamOnNewPipeline(
+ ClientSocketHandle* connection,
+ const SSLConfig& used_ssl_config,
+ const ProxyInfo& used_proxy_info,
+ const BoundNetLog& net_log,
+ bool was_npn_negotiated) OVERRIDE;
+
+ virtual HttpPipelinedStream* CreateStreamOnExistingPipeline() OVERRIDE;
+
+ virtual bool IsExistingPipelineAvailable() const OVERRIDE;
+
+ // HttpPipelinedConnection::Delegate interface
+
+ // Called when a pipelined connection completes a request. Adds a pending
+ // request to the pipeline if the pipeline is still usable.
+ virtual void OnPipelineHasCapacity(
+ HttpPipelinedConnection* pipeline) OVERRIDE;
+
+ virtual void OnPipelineFeedback(
+ HttpPipelinedConnection* pipeline,
+ HttpPipelinedConnection::Feedback feedback) OVERRIDE;
+
+ virtual const HostPortPair& origin() const OVERRIDE;
+
+ // Returns the maximum number of in-flight pipelined requests we'll allow on a
+ // single connection.
+ NET_EXPORT_PRIVATE static int max_pipeline_depth() { return 3; }
+
+ private:
+ struct PipelineInfo {
+ PipelineInfo();
+
+ int num_successes;
+ };
+ typedef std::map<HttpPipelinedConnection*, PipelineInfo> PipelineInfoMap;
+
+ // Called when a pipeline is empty and there are no pending requests. Closes
+ // the connection.
+ void OnPipelineEmpty(HttpPipelinedConnection* pipeline);
+
+ // Adds the next pending request to the pipeline if it's still usuable.
+ void AddRequestToPipeline(HttpPipelinedConnection* connection);
+
+ // Returns the current pipeline capacity based on |capability_|. This should
+ // not be called if |capability_| is INCAPABLE.
+ int GetPipelineCapacity() const;
+
+ HttpPipelinedHost::Delegate* delegate_;
+ const HostPortPair origin_;
+ PipelineInfoMap pipelines_;
+ scoped_ptr<HttpPipelinedConnection::Factory> factory_;
+ Capability capability_;
+
+ DISALLOW_COPY_AND_ASSIGN(HttpPipelinedHostImpl);
+};
+
+} // namespace net
+
+#endif // NET_HTTP_HTTP_PIPELINED_HOST_IMPL_H_
« no previous file with comments | « net/http/http_pipelined_host.cc ('k') | net/http/http_pipelined_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698