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

Unified Diff: net/nqe/network_quality_estimator.cc

Issue 2720513002: Change the string representation of Slow2G ECT (Closed)
Patch Set: ryansturm comments Created 3 years, 10 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 | « net/nqe/effective_connection_type_unittest.cc ('k') | net/nqe/network_quality_estimator_params.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/nqe/network_quality_estimator.cc
diff --git a/net/nqe/network_quality_estimator.cc b/net/nqe/network_quality_estimator.cc
index cc8428189ec18610cc17bb7174a02a74054882c3..abe4e1e54f406bbed6fa578820998ba59ac50a11 100644
--- a/net/nqe/network_quality_estimator.cc
+++ b/net/nqe/network_quality_estimator.cc
@@ -191,11 +191,12 @@ void RecordEffectiveConnectionTypeAccuracy(
int32_t metric,
base::TimeDelta measuring_duration,
EffectiveConnectionType observed_effective_connection_type) {
- const std::string histogram_name = base::StringPrintf(
- "%s.EstimatedObservedDiff.%s.%d.%s", prefix,
- metric >= 0 ? "Positive" : "Negative",
- static_cast<int32_t>(measuring_duration.InSeconds()),
- GetNameForEffectiveConnectionType(observed_effective_connection_type));
+ const std::string histogram_name =
+ base::StringPrintf("%s.EstimatedObservedDiff.%s.%d.%s", prefix,
+ metric >= 0 ? "Positive" : "Negative",
+ static_cast<int32_t>(measuring_duration.InSeconds()),
+ DeprecatedGetNameForEffectiveConnectionType(
+ observed_effective_connection_type));
base::HistogramBase* histogram = base::Histogram::FactoryGet(
histogram_name, 0, EFFECTIVE_CONNECTION_TYPE_LAST,
« no previous file with comments | « net/nqe/effective_connection_type_unittest.cc ('k') | net/nqe/network_quality_estimator_params.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698