Commit c696644e authored by Tarun Bansal's avatar Tarun Bansal Committed by Commit Bot

Remove unused local histogram in NQE.

Remove unused local histogram
NQE.CongestionAnalyzer.CountInflightRequestsForPeakQueueingDelay.*
in NQE.

It's also using the local macro incorrectly
causing crashes. Macro histograms are expected to
use static name (and not a runtime generated name).

Change-Id: I196449d759cff2eeb1919f1bfb39af66e7225c1a
TBR: ryansturm@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2159405
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: default avatarRyan Sturm <ryansturm@chromium.org>
Reviewed-by: default avatarTarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#760972}
parent d48fa763
...@@ -348,22 +348,6 @@ void NetworkCongestionAnalyzer::FinalizeCurrentMeasurementPeriod() { ...@@ -348,22 +348,6 @@ void NetworkCongestionAnalyzer::FinalizeCurrentMeasurementPeriod() {
count_inflight_requests_for_peak_queueing_delay_; count_inflight_requests_for_peak_queueing_delay_;
} }
size_t peak_queueing_delay_level =
ComputePeakQueueingDelayLevel(peak_queueing_delay_);
DCHECK_GE(kQueueingDelayLevelMaxVal, peak_queueing_delay_level);
if (peak_queueing_delay_level >= kQueueingDelayLevelMinVal &&
peak_queueing_delay_level <= kQueueingDelayLevelMaxVal) {
// Records the count of in-flight requests causing the peak queueing delay
// within the current measurement period. These samples are bucketized
// into 10 peak queueing delay levels.
LOCAL_HISTOGRAM_COUNTS_100(
"NQE.CongestionAnalyzer.CountInflightRequestsForPeakQueueingDelay."
"Level" +
base::NumberToString(peak_queueing_delay_level),
count_inflight_requests_for_peak_queueing_delay_);
}
UpdateRequestsCountAndPeakQueueingDelayMapping(); UpdateRequestsCountAndPeakQueueingDelayMapping();
} }
......
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