Commit 3e4d1fa0 authored by ftirelo's avatar ftirelo Committed by Commit bot

Report FoundUwSReadError only once per Software Reporter run.

This was introduced by https://codereview.chromium.org/2347753002, during
a code refactoring. The bug is not critical, since we don't look at this
data with much frequency, but we should revert it to the intended behavior.

BUG=676031

Review-Url: https://codereview.chromium.org/2593783002
Cr-Commit-Position: refs/heads/master@{#439889}
parent 360dee28
...@@ -272,12 +272,11 @@ class UMAHistogramReporter { ...@@ -272,12 +272,11 @@ class UMAHistogramReporter {
} else { } else {
parse_error = true; parse_error = true;
} }
// Clean up the old value.
reporter_key.DeleteValue(kFoundUwsValueName);
RecordBooleanHistogram(kFoundUwsReadErrorMetricName, parse_error);
} }
// Clean up the old value.
reporter_key.DeleteValue(kFoundUwsValueName);
RecordBooleanHistogram(kFoundUwsReadErrorMetricName, parse_error);
} }
// Reports to UMA the memory usage of the software reporter tool as reported // Reports to UMA the memory usage of the software reporter tool as reported
......
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