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

Unified Diff: chrome/browser/net/chrome_network_delegate_unittest.cc

Issue 2338563003: Record UMA for total data usage consumed by Chromium (Closed)
Patch Set: rebased, addressed gayane comment Created 4 years, 3 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: chrome/browser/net/chrome_network_delegate_unittest.cc
diff --git a/chrome/browser/net/chrome_network_delegate_unittest.cc b/chrome/browser/net/chrome_network_delegate_unittest.cc
index 27f3f9aa9750cc4f7535f750cefe10bb2d59ed23..0fa6f827e71ef6d4a74f410cf12a09b0bf2bb67b 100644
--- a/chrome/browser/net/chrome_network_delegate_unittest.cc
+++ b/chrome/browser/net/chrome_network_delegate_unittest.cc
@@ -8,6 +8,7 @@
#include <memory>
#include <utility>
+#include <vector>
#include "base/command_line.h"
#include "base/macros.h"
@@ -194,6 +195,21 @@ class ChromeNetworkDelegateTest : public testing::Test {
std::unique_ptr<net::TestURLRequestContext> context_;
};
+// Test that the total data use consumed by Chrome is recorded correctly.
+TEST_F(ChromeNetworkDelegateTest, TotalDataUseMeasurementTest) {
+ Initialize();
+ base::HistogramTester histogram_tester;
+
+ // A query from a user without redirection.
+ RequestURL(context(), socket_factory(), true, false);
+ std::vector<base::Bucket> buckets =
+ histogram_tester.GetAllSamples("DataUse.BytesSent.Delegate");
+ EXPECT_FALSE(buckets.empty());
+
+ buckets = histogram_tester.GetAllSamples("DataUse.BytesReceived.Delegate");
+ EXPECT_FALSE(buckets.empty());
+}
+
// This function tests data use measurement for requests by services. it makes a
// query which is similar to a query of a service, so it should affect
// DataUse.TrafficSize.System.Dimensions and DataUse.MessageSize.ServiceName
« no previous file with comments | « chrome/browser/net/chrome_network_delegate.cc ('k') | components/data_use_measurement/content/data_use_measurement.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698