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

Unified Diff: net/http/http_cache.cc

Issue 4013: Port more unit tests in net/http/ (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: Created 12 years, 3 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
« no previous file with comments | « net/base/x509_certificate.h ('k') | net/http/http_cache_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_cache.cc
===================================================================
--- net/http/http_cache.cc (revision 2439)
+++ net/http/http_cache.cc (working copy)
@@ -1034,6 +1034,7 @@
response_info->headers = new HttpResponseHeaders(pickle, &iter);
DCHECK(response_info->headers->response_code() != -1);
+#if defined(OS_WIN)
// read ssl-info
if (flags & RESPONSE_INFO_HAS_CERT) {
response_info->ssl_info.cert =
@@ -1051,6 +1052,10 @@
return false;
response_info->ssl_info.security_bits = security_bits;
}
+#else
+ // TODO(port): Just enable above when SSL stuff is ported.
+ NOTIMPLEMENTED();
+#endif
// read vary-data
if (flags & RESPONSE_INFO_HAS_VARY_DATA) {
@@ -1082,12 +1087,17 @@
response_info->headers->Persist(&pickle, skip_transient_headers);
+#if defined(OS_WIN)
if (response_info->ssl_info.cert) {
response_info->ssl_info.cert->Persist(&pickle);
pickle.WriteInt(response_info->ssl_info.cert_status);
}
if (response_info->ssl_info.security_bits != -1)
pickle.WriteInt(response_info->ssl_info.security_bits);
+#else
+ // TODO(port): Just enable above when SSL stuff is ported.
+ NOTIMPLEMENTED();
+#endif
if (response_info->vary_data.is_valid())
response_info->vary_data.Persist(&pickle);
« no previous file with comments | « net/base/x509_certificate.h ('k') | net/http/http_cache_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698