Add UMA Precache.Freshness.Prefetch

components/precache/content/precache_manager.cc

Index: components/precache/content/precache_manager.cc
diff --git a/components/precache/content/precache_manager.cc b/components/precache/content/precache_manager.cc
index 9a81fbb032b3e47ee7de0748794ec3c30ea0c096..81e09844fe91e1e7d00c127d6a4794ad88f476a4 100644
--- a/components/precache/content/precache_manager.cc
+++ b/components/precache/content/precache_manager.cc
@@ -26,6 +26,7 @@
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/storage_partition.h"
 #include "net/base/network_change_notifier.h"
+#include "net/http/http_response_info.h"

[bengr, 2016/07/21 00:03:27]

Remove this include. You will forward declare in the .h

[jamartin, 2016/07/25 23:23:07]

Done.

 
 using content::BrowserThread;
 
@@ -219,12 +220,12 @@ void PrecacheManager::UpdatePrecacheMetricsAndState(
     const GURL& referrer,
     const base::TimeDelta& latency,
     const base::Time& fetch_time,
+    const net::HttpResponseInfo& info,
     int64_t size,
-    bool was_cached,
     bool is_user_traffic) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
-  RecordStatsForFetch(url, referrer, latency, fetch_time, size, was_cached);
+  RecordStatsForFetch(url, referrer, latency, fetch_time, info, size);
   if (is_user_traffic && IsPrecaching())
     CancelPrecaching();
 }
@@ -233,8 +234,8 @@ void PrecacheManager::RecordStatsForFetch(const GURL& url,
                                           const GURL& referrer,
                                           const base::TimeDelta& latency,
                                           const base::Time& fetch_time,
-                                          int64_t size,
-                                          bool was_cached) {
+                                          const net::HttpResponseInfo& info,
+                                          int64_t size) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   if (size == 0 || url.is_empty() || !url.SchemeIsHTTPOrHTTPS()) {
@@ -246,17 +247,16 @@ void PrecacheManager::RecordStatsForFetch(const GURL& url,
     return;
 
   history_service_->HostRankIfAvailable(
-      referrer,
-      base::Bind(&PrecacheManager::RecordStatsForFetchInternal, AsWeakPtr(),
-                 url, latency, fetch_time, size, was_cached));
+      referrer, base::Bind(&PrecacheManager::RecordStatsForFetchInternal,
+                           AsWeakPtr(), url, latency, fetch_time, info, size));
 }
 
 void PrecacheManager::RecordStatsForFetchInternal(
     const GURL& url,
     const base::TimeDelta& latency,
     const base::Time& fetch_time,
+    const net::HttpResponseInfo& info,
     int64_t size,
-    bool was_cached,
     int host_rank) {
   if (is_precaching_) {
     // Assume that precache is responsible for all requests made while
@@ -268,7 +268,7 @@ void PrecacheManager::RecordStatsForFetchInternal(
         BrowserThread::DB, FROM_HERE,
         base::Bind(&PrecacheDatabase::RecordURLPrefetch,
                    base::Unretained(precache_database_.get()), url, latency,
-                   fetch_time, size, was_cached));
+                   fetch_time, info, size));
   } else {
     bool is_connection_cellular =
         net::NetworkChangeNotifier::IsConnectionCellular(
@@ -278,8 +278,7 @@ void PrecacheManager::RecordStatsForFetchInternal(
         BrowserThread::DB, FROM_HERE,
         base::Bind(&PrecacheDatabase::RecordURLNonPrefetch,
                    base::Unretained(precache_database_.get()), url, latency,
-                   fetch_time, size, was_cached, host_rank,
-                   is_connection_cellular));
+                   fetch_time, info, size, host_rank, is_connection_cellular));
   }
 }