Add UMA to determine how often we offline a page with previews.

chrome/browser/loader/chrome_navigation_data.h

Index: chrome/browser/loader/chrome_navigation_data.h
diff --git a/chrome/browser/loader/chrome_navigation_data.h b/chrome/browser/loader/chrome_navigation_data.h
index 30268fa656073b351c38f56e57fbc10abb81d93a..ace54be60f950e0657ff476d7f481d3443e2f8a1 100644
--- a/chrome/browser/loader/chrome_navigation_data.h
+++ b/chrome/browser/loader/chrome_navigation_data.h
@@ -9,11 +9,9 @@
 
 #include "base/macros.h"
 #include "base/supports_user_data.h"
+#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_data.h"

[RyanSturm, 2017/06/02 20:50:33]

Any reason for switching forward declare to header? Is it needed for the
std::move so that the destructor is declared?

[Pete Williamson, 2017/06/02 21:40:32]

It was a different compile error:
 error: invalid application of 'sizeof' to an incomplete type
'data_reduction_proxy::DataReductionProxyData'

 #include "content/public/browser/navigation_data.h"
-
-namespace data_reduction_proxy {
-class DataReductionProxyData;
-}
+#include "content/public/common/previews_state.h"
 
 namespace net {
 class URLRequest;
@@ -43,6 +41,11 @@ class ChromeNavigationData : public content::NavigationData,
     return data_reduction_proxy_data_.get();
   }
 
+  content::PreviewsState previews_state() { return previews_state_; }
+  void set_previews_state(content::PreviewsState previews_state) {
+    previews_state_ = previews_state;
+  }
+
   static ChromeNavigationData* GetDataAndCreateIfNecessary(
       net::URLRequest* request);
 
@@ -50,6 +53,7 @@ class ChromeNavigationData : public content::NavigationData,
   // Manages the lifetime of optional DataReductionProxy information.
   std::unique_ptr<data_reduction_proxy::DataReductionProxyData>
       data_reduction_proxy_data_;
+  content::PreviewsState previews_state_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeNavigationData);
 };