Add UMA Precache.Freshness.Prefetch

components/precache/content/precache_manager.h

Index: components/precache/content/precache_manager.h
diff --git a/components/precache/content/precache_manager.h b/components/precache/content/precache_manager.h
index 259ced7678a585fbac5d62d569a4efd0fe06eebf..755f39c28d0eb35fa2cef02f02412b9e4a15f52a 100644
--- a/components/precache/content/precache_manager.h
+++ b/components/precache/content/precache_manager.h
@@ -21,6 +21,7 @@
 #include "components/history/core/browser/history_types.h"
 #include "components/keyed_service/core/keyed_service.h"
 #include "components/precache/core/precache_fetcher.h"
+#include "net/http/http_response_info.h"

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

Forward declare instead.

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

Done.

Nice! This should get rid of the change in DEPS and thus one reviewer less :-)

 #include "url/gurl.h"
 
 namespace base {
@@ -108,8 +109,8 @@ class PrecacheManager : public KeyedService,
                                      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);
 
  private:
@@ -159,16 +160,16 @@ class PrecacheManager : public KeyedService,
                            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);
 
   // Update precache-related metrics in response to a URL being fetched. Called
   // by RecordStatsForFetch() by way of an asynchronous HistoryService callback.
   void 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);
 
   // The browser context that owns this PrecacheManager.