Commit 23be9745 authored by timvolodine's avatar timvolodine Committed by Commit bot

Fix number of buckets in AsyncDNS.ServerCount and ServerFailureIndex UMA histograms.

Fix number of buckets in AsyncDNS.ServerCount and SyncDNS.ServerFailureIndex UMA
histograms. Currently the value 9 falls in the bucket with value 8 which is
confusing. Results can be checked on uma.googleplex.com.

BUG=

Review URL: https://codereview.chromium.org/551873002

Cr-Commit-Position: refs/heads/master@{#296950}
parent f4b6a28e
......@@ -89,7 +89,7 @@ DnsSession::DnsSession(const DnsConfig& config,
server_index_(0) {
socket_pool_->Initialize(&config_.nameservers, net_log);
UMA_HISTOGRAM_CUSTOM_COUNTS(
"AsyncDNS.ServerCount", config_.nameservers.size(), 0, 10, 10);
"AsyncDNS.ServerCount", config_.nameservers.size(), 0, 10, 11);
for (size_t i = 0; i < config_.nameservers.size(); ++i) {
server_stats_.push_back(new ServerStats(config_.timeout,
rtt_buckets_.Pointer()));
......@@ -139,7 +139,7 @@ unsigned DnsSession::NextGoodServerIndex(unsigned server_index) {
void DnsSession::RecordServerFailure(unsigned server_index) {
UMA_HISTOGRAM_CUSTOM_COUNTS(
"AsyncDNS.ServerFailureIndex", server_index, 0, 10, 10);
"AsyncDNS.ServerFailureIndex", server_index, 0, 10, 11);
++(server_stats_[server_index]->last_failure_count);
server_stats_[server_index]->last_failure = base::Time::Now();
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment