Exposing CacheEntryStatus (former TransactionPattern) via UrlRequest

net/http/http_response_info.cc

Index: net/http/http_response_info.cc
diff --git a/net/http/http_response_info.cc b/net/http/http_response_info.cc
index b0c23e032880bc4afa0222bb70139e304466aa2f..fe973921b4ae1dfd91910dda1c923f2f9d79bc9e 100644
--- a/net/http/http_response_info.cc
+++ b/net/http/http_response_info.cc
@@ -106,6 +106,7 @@ enum {
 
 HttpResponseInfo::HttpResponseInfo()
     : was_cached(false),
+      cache_info(CacheInfo::UNDEFINED),
       server_data_unavailable(false),
       network_accessed(false),
       was_fetched_via_spdy(false),
@@ -118,6 +119,7 @@ HttpResponseInfo::HttpResponseInfo()
 
 HttpResponseInfo::HttpResponseInfo(const HttpResponseInfo& rhs)
     : was_cached(rhs.was_cached),
+      cache_info(rhs.cache_info),
       server_data_unavailable(rhs.server_data_unavailable),
       network_accessed(rhs.network_accessed),
       was_fetched_via_spdy(rhs.was_fetched_via_spdy),
@@ -144,6 +146,7 @@ HttpResponseInfo::~HttpResponseInfo() {
 
 HttpResponseInfo& HttpResponseInfo::operator=(const HttpResponseInfo& rhs) {
   was_cached = rhs.was_cached;
+  cache_info = rhs.cache_info;
   server_data_unavailable = rhs.server_data_unavailable;
   network_accessed = rhs.network_accessed;
   was_fetched_via_spdy = rhs.was_fetched_via_spdy;
@@ -188,6 +191,7 @@ bool HttpResponseInfo::InitFromPickle(const base::Pickle& pickle,
     return false;
   request_time = Time::FromInternalValue(time_val);
   was_cached = true;  // Set status to show cache resurrection.
+  cache_info = CacheInfo::ENTRY_USED;

[jkarlin, 2016/06/30 19:01:12]

It's too early to set ENTRY_USED here. HttpCacheTransaction will set this.
Suggest deleting this line.

[jamartin, 2016/07/04 17:12:26]

Done.

 
   // Read response-time
   if (!iter.ReadInt64(&time_val))