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

Unified Diff: chrome/browser/loader/chrome_navigation_data.cc

Issue 2916253002: Add UMA to determine how often we offline a page with previews. (Closed)
Patch Set: Histograms fix Created 3 years, 6 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
Index: chrome/browser/loader/chrome_navigation_data.cc
diff --git a/chrome/browser/loader/chrome_navigation_data.cc b/chrome/browser/loader/chrome_navigation_data.cc
index c6069ab2c01f04e9d937faa05fe32d2b2ab3b20d..76e9c6b6eaf61464e26d783392ca88da9023232c 100644
--- a/chrome/browser/loader/chrome_navigation_data.cc
+++ b/chrome/browser/loader/chrome_navigation_data.cc
@@ -5,13 +5,13 @@
#include "chrome/browser/loader/chrome_navigation_data.h"
#include "base/memory/ptr_util.h"
-#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_data.h"
#include "net/url_request/url_request.h"
const void* const kChromeNavigationDataUserDataKey =
&kChromeNavigationDataUserDataKey;
-ChromeNavigationData::ChromeNavigationData() {}
+ChromeNavigationData::ChromeNavigationData()
+ : previews_state_(content::PreviewsTypes::PREVIEWS_UNSPECIFIED) {}
ChromeNavigationData::~ChromeNavigationData() {}
@@ -31,7 +31,9 @@ ChromeNavigationData* ChromeNavigationData::GetDataAndCreateIfNecessary(
std::unique_ptr<content::NavigationData> ChromeNavigationData::Clone() const {
std::unique_ptr<ChromeNavigationData> copy(new ChromeNavigationData());
- if (data_reduction_proxy_data_)
+ if (data_reduction_proxy_data_) {
copy->SetDataReductionProxyData(data_reduction_proxy_data_->DeepCopy());
+ }
+ copy->previews_state_ = previews_state_;
return std::move(copy);
}
« no previous file with comments | « chrome/browser/loader/chrome_navigation_data.h ('k') | chrome/browser/loader/chrome_resource_dispatcher_host_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698