diff options
author | eroman@chromium.org <eroman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-04 22:51:53 +0000 |
---|---|---|
committer | eroman@chromium.org <eroman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-04 22:51:53 +0000 |
commit | cb4ff9d7374c09b4ff0c8e257e15b81bb7bb42ed (patch) | |
tree | c21d92eccc0229847c72b37b370b31440783df39 /net/http | |
parent | a57337558a95b22d2808ff890e270cfee22985a8 (diff) | |
download | chromium_src-cb4ff9d7374c09b4ff0c8e257e15b81bb7bb42ed.zip chromium_src-cb4ff9d7374c09b4ff0c8e257e15b81bb7bb42ed.tar.gz chromium_src-cb4ff9d7374c09b4ff0c8e257e15b81bb7bb42ed.tar.bz2 |
Re-enabled URLRequestTestHTTP.BasicAuth.
The problem was the test was expecting |response_time| of cached responses to be the timestamp of the original request, rather than the timestamp of the last cache validation.
Also updated the header to clarify what |request_time| and |response_time| correspond to in the cached cases.
BUG=http://crbug.com/20594
Review URL: http://codereview.chromium.org/197030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@25540 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http')
-rw-r--r-- | net/http/http_response_info.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/http/http_response_info.h b/net/http/http_response_info.h index 7cbe13c..62ac2f4 100644 --- a/net/http/http_response_info.h +++ b/net/http/http_response_info.h @@ -31,11 +31,11 @@ class HttpResponseInfo { bool was_cached; // The time at which the request was made that resulted in this response. - // For cached responses, this time could be "far" in the past. + // For cached responses, this is the last time the cache entry was validated. base::Time request_time; // The time at which the response headers were received. For cached - // responses, this time could be "far" in the past. + // this is the last time the cache entry was validated. base::Time response_time; // If the response headers indicate a 401 or 407 failure, then this structure |