Record UMA for total data usage consumed by Chromium

chrome/browser/net/chrome_network_delegate.cc

Index: chrome/browser/net/chrome_network_delegate.cc
diff --git a/chrome/browser/net/chrome_network_delegate.cc b/chrome/browser/net/chrome_network_delegate.cc
index 6dd36c0cb099a74903ac99bd6ab1c0025f028a76..64af726ee1c15390bf59ef79f0975119ad624de6 100644
--- a/chrome/browser/net/chrome_network_delegate.cc
+++ b/chrome/browser/net/chrome_network_delegate.cc
@@ -364,11 +364,13 @@ void ChromeNetworkDelegate::OnNetworkBytesReceived(net::URLRequest* request,
 #endif  // defined(ENABLE_TASK_MANAGER)
 
   ReportDataUsageStats(request, 0 /* tx_bytes */, bytes_received);
+  data_use_measurement_.OnNetworkBytesReceived(*request, bytes_received);

[sclittle, 2016/09/14 21:28:13]

Could you explain (e.g. in a comment in the code) why you can't just register
data_use_measurement_ as an observer on the DataUseAggregator? E.g., what's
different about this use case?

[tbansal1, 2016/09/15 17:46:04]

I want to record the data use by both the delegate and at TrafficStats layer.
DataUseAggregator provides measurements only at one of the two layers. Also, I
want to record overall data use (including incognito) which DataUseAggregator
does not provide. I have added comments to d_u_measurement.cc file.

 }
 
 void ChromeNetworkDelegate::OnNetworkBytesSent(net::URLRequest* request,
                                                int64_t bytes_sent) {
   ReportDataUsageStats(request, bytes_sent, 0 /* rx_bytes */);
+  data_use_measurement_.OnNetworkBytesSent(*request, bytes_sent);
 }
 
 void ChromeNetworkDelegate::OnCompleted(net::URLRequest* request,