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

Unified Diff: net/quic/quic_in_memory_cache.h

Issue 337093003: QuicServer: Use Chrome's header parsing rather than Balsa (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@quic_server_3
Patch Set: (mark StringPieceUtils as copied, not added) Created 6 years, 6 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/quic/quic_in_memory_cache.h
diff --git a/net/quic/quic_in_memory_cache.h b/net/quic/quic_in_memory_cache.h
index 32dc8d4cf0c587af01006a5b5ceaa25eb73217a2..45ed200ddde42fdcda31f7dbb2aad2ee80d24a5b 100644
--- a/net/quic/quic_in_memory_cache.h
+++ b/net/quic/quic_in_memory_cache.h
@@ -11,12 +11,12 @@
#include "base/file_util.h"
#include "base/memory/singleton.h"
#include "base/strings/string_piece.h"
-#include "net/base/net_export.h"
-#include "net/tools/balsa/balsa_frame.h"
-#include "net/tools/balsa/balsa_headers.h"
-#include "net/tools/balsa/noop_balsa_visitor.h"
+#include "net/http/http_request_headers.h"
+#include "net/http/http_response_headers.h"
+#include "net/quic/http_request_line.h"
-template <typename T> struct DefaultSingletonTraits;
+template <typename T>
+struct DefaultSingletonTraits;
namespace net {
@@ -42,25 +42,23 @@ class QuicInMemoryCache {
// Container for response header/body pairs.
class Response {
public:
- Response() : response_type_(REGULAR_RESPONSE) {}
- ~Response() {}
+ Response();
+ ~Response();
SpecialResponseType response_type() const { return response_type_; }
- const BalsaHeaders& headers() const { return headers_; }
+ scoped_refptr<HttpResponseHeaders> headers() const { return headers_; }
const base::StringPiece body() const { return base::StringPiece(body_); }
private:
friend class QuicInMemoryCache;
- void set_headers(const BalsaHeaders& headers) {
- headers_.CopyFrom(headers);
- }
- void set_body(base::StringPiece body) {
- body.CopyToString(&body_);
+ void set_headers(scoped_refptr<HttpResponseHeaders> headers) {
+ headers_ = headers;
}
+ void set_body(base::StringPiece body) { body.CopyToString(&body_); }
SpecialResponseType response_type_;
- BalsaHeaders headers_;
+ scoped_refptr<HttpResponseHeaders> headers_;
std::string body_;
DISALLOW_COPY_AND_ASSIGN(Response);
@@ -72,7 +70,7 @@ class QuicInMemoryCache {
// Retrieve a response from this cache for a given request.
// If no appropriate response exists, NULL is returned.
// Currently, responses are selected based on request URI only.
- const Response* GetResponse(const BalsaHeaders& request_headers) const;
+ const Response* GetResponse(const HttpRequestLine& request_line) const;
// Adds a simple response to the cache. The response headers will
// only contain the "content-length" header with the lenght of |body|.
@@ -84,8 +82,8 @@ class QuicInMemoryCache {
base::StringPiece body);
// Add a response to the cache.
- void AddResponse(const BalsaHeaders& request_headers,
- const BalsaHeaders& response_headers,
+ void AddResponse(const HttpRequestLine& request_line,
+ scoped_refptr<HttpResponseHeaders> response_headers,
base::StringPiece response_body);
// Simulate a special behavior at a particular path.
@@ -106,7 +104,7 @@ class QuicInMemoryCache {
void Initialize();
- std::string GetKey(const BalsaHeaders& response_headers) const;
+ std::string GetKey(const HttpRequestLine& request_line) const;
// Cached responses.
ResponseMap responses_;

Powered by Google App Engine
This is Rietveld 408576698