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

Unified Diff: components/cronet/android/cronet_url_request_adapter.cc

Issue 2391423002: Implement getSentBytesCount() in Cronet UrlRequest metrics (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « no previous file | components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/android/cronet_url_request_adapter.cc
diff --git a/components/cronet/android/cronet_url_request_adapter.cc b/components/cronet/android/cronet_url_request_adapter.cc
index 0ef0415210e6438cb3ba7e7d5f82375b9b31fbf8..e0820aa46b6db0584abd0cd4147aaa33b0d02f9b 100644
--- a/components/cronet/android/cronet_url_request_adapter.cc
+++ b/components/cronet/android/cronet_url_request_adapter.cc
@@ -430,8 +430,8 @@ void CronetURLRequestAdapter::MaybeReportMetrics(JNIEnv* env) const {
ConvertTime(metrics.push_end, start_ticks, start_time),
ConvertTime(metrics.receive_headers_end, start_ticks, start_time),
ConvertTime(base::TimeTicks::Now(), start_ticks, start_time),
- // TODO(mgersh): report total bytes sent
- metrics.socket_reused, 0, url_request_->GetTotalReceivedBytes());
+ metrics.socket_reused, url_request_->GetTotalSentBytes(),
+ url_request_->GetTotalReceivedBytes());
}
net::URLRequest* CronetURLRequestAdapter::GetURLRequestForTesting() {
« no previous file with comments | « no previous file | components/cronet/android/test/javatests/src/org/chromium/net/QuicTest.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698