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

Unified Diff: components/data_use_measurement/core/data_use_measurement.cc

Issue 2614203002: Record data use of user traffic by different core page transition types (Closed)
Patch Set: rebased Created 3 years, 9 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: components/data_use_measurement/core/data_use_measurement.cc
diff --git a/components/data_use_measurement/core/data_use_measurement.cc b/components/data_use_measurement/core/data_use_measurement.cc
index 2c7b0b3f0e66b83d3de1b752519916a8a4d74f45..0e2794cf1902ff550dfec6b2a861e4027719be8f 100644
--- a/components/data_use_measurement/core/data_use_measurement.cc
+++ b/components/data_use_measurement/core/data_use_measurement.cc
@@ -169,6 +169,7 @@ void DataUseMeasurement::OnCompleted(const net::URLRequest& request,
// of redirected requests.
UpdateDataUsePrefs(request);
ReportServicesMessageSizeUMA(request);
+ RecordPageTransitionUMA(request);
#if defined(OS_ANDROID)
MaybeRecordNetworkBytesOS();
#endif
@@ -392,7 +393,7 @@ void DataUseMeasurement::RecordTabStateHistogram(
TrafficDirection dir,
DataUseUserData::AppState app_state,
bool is_tab_visible,
- int64_t bytes) {
+ int64_t bytes) const {
if (app_state == DataUseUserData::UNKNOWN)
return;
@@ -444,4 +445,16 @@ void DataUseMeasurement::RecordContentTypeHistogram(
}
}
+void DataUseMeasurement::RecordPageTransitionUMA(
+ const net::URLRequest& request) const {
+ if (!url_request_classifier_->IsUserRequest(request))
+ return;
+
+ const DataUseRecorder* recorder = ascriber_->GetDataUseRecorder(request);
+ if (recorder) {
+ url_request_classifier_->RecordPageTransitionUMA(
+ recorder->page_transition(), request.GetTotalReceivedBytes());
+ }
+}
+
} // namespace data_use_measurement

Powered by Google App Engine
This is Rietveld 408576698