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

Side by Side Diff: net/http/http_vary_data.h

Issue 1604011: Use HttpRequestHeaders for extra_headers. (Closed)
Patch Set: Address eroman comments. Created 10 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 unified diff | Download patch
« no previous file with comments | « net/http/http_util.cc ('k') | net/http/http_vary_data.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef NET_HTTP_HTTP_VARY_DATA_H__ 5 #ifndef NET_HTTP_HTTP_VARY_DATA_H__
6 #define NET_HTTP_HTTP_VARY_DATA_H__ 6 #define NET_HTTP_HTTP_VARY_DATA_H__
7 7
8 #include "base/md5.h" 8 #include "base/md5.h"
9 9
10 class Pickle; 10 class Pickle;
11 11
12 namespace net { 12 namespace net {
13 13
14 class HttpRequestInfo; 14 struct HttpRequestInfo;
15 class HttpResponseHeaders; 15 class HttpResponseHeaders;
16 16
17 // Used to implement the HTTP/1.1 Vary header. This class contains a MD5 hash 17 // Used to implement the HTTP/1.1 Vary header. This class contains a MD5 hash
18 // over the request headers indicated by a Vary header. 18 // over the request headers indicated by a Vary header.
19 // 19 //
20 // While RFC 2616 requires strict request header comparisons, it is much 20 // While RFC 2616 requires strict request header comparisons, it is much
21 // cheaper to store a MD5 sum, which should be sufficient. Storing a hash also 21 // cheaper to store a MD5 sum, which should be sufficient. Storing a hash also
22 // avoids messy privacy issues as some of the request headers could hold 22 // avoids messy privacy issues as some of the request headers could hold
23 // sensitive data (e.g., cookies). 23 // sensitive data (e.g., cookies).
24 // 24 //
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 // A digested version of the request headers corresponding to the Vary header. 75 // A digested version of the request headers corresponding to the Vary header.
76 MD5Digest request_digest_; 76 MD5Digest request_digest_;
77 77
78 // True when request_digest_ contains meaningful data. 78 // True when request_digest_ contains meaningful data.
79 bool is_valid_; 79 bool is_valid_;
80 }; 80 };
81 81
82 } // namespace net 82 } // namespace net
83 83
84 #endif // NET_HTTP_HTTP_VARY_DATA_H__ 84 #endif // NET_HTTP_HTTP_VARY_DATA_H__
OLDNEW
« no previous file with comments | « net/http/http_util.cc ('k') | net/http/http_vary_data.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698