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

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

Issue 2950603003: Minor refactor to get rid of a friend association (Closed)
Patch Set: 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: components/data_use_measurement/core/data_use_recorder.cc
diff --git a/components/data_use_measurement/core/data_use_recorder.cc b/components/data_use_measurement/core/data_use_recorder.cc
index 732796e134c5f00651af27bcab5e533e367537de..0967f358b3a0621747526b651e22b1969a43d072 100644
--- a/components/data_use_measurement/core/data_use_recorder.cc
+++ b/components/data_use_measurement/core/data_use_recorder.cc
@@ -33,24 +33,12 @@ void DataUseRecorder::OnBeforeUrlRequest(net::URLRequest* request) {}
void DataUseRecorder::OnNetworkBytesReceived(net::URLRequest* request,
int64_t bytes_received) {
- data_use_.total_bytes_received_ += bytes_received;
+ data_use_.IncrementTotalBytes(bytes_received, 0);
}
void DataUseRecorder::OnNetworkBytesSent(net::URLRequest* request,
int64_t bytes_sent) {
- data_use_.total_bytes_sent_ += bytes_sent;
-}
-
-void DataUseRecorder::AddPendingDataSource(void* source) {
- pending_data_sources_.insert(source);
-}
-
-bool DataUseRecorder::HasPendingDataSource(void* source) {
- return pending_data_sources_.find(source) != pending_data_sources_.end();
-}
-
-void DataUseRecorder::RemovePendingDataSource(void* source) {
- pending_data_sources_.erase(source);
+ data_use_.IncrementTotalBytes(0, bytes_sent);
}
void DataUseRecorder::MergeFrom(DataUseRecorder* other) {

Powered by Google App Engine
This is Rietveld 408576698