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

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: Addressed comments Created 3 years, 11 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 6508f99ade7c1e38116388d23aa7e36c89949ba4..3283666395c9177a7fbe8a11d8ba425a1dc9a2fd 100644
--- a/components/data_use_measurement/core/data_use_measurement.cc
+++ b/components/data_use_measurement/core/data_use_measurement.cc
@@ -143,6 +143,7 @@ void DataUseMeasurement::OnCompleted(const net::URLRequest& request,
// of redirected requests.
UpdateDataUsePrefs(request);
ReportServicesMessageSizeUMA(request);
+ RecordPageTransitionUMA(request);
#if defined(OS_ANDROID)
MaybeRecordNetworkBytesOS();
#endif
@@ -347,7 +348,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;
@@ -363,4 +364,16 @@ void DataUseMeasurement::RecordTabStateHistogram(
RecordUMAHistogramCount(histogram_name, bytes);
}
+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