[TaskScheduler] Do not log 0 to TaskScheduler.NumTasksBetweenWaits histogram on detach.
Doing so accesses |outer_| which was unsafe after invoking Cleanup().
While this will be made safe with https://chromium-review.googlesource.com/c/chromium/src/+/931547
we had noticed this metric being wrong in the past but had postponed
cleaning up to avoid perturbing metrics as part of a bigger change.
Now is the time to drop this report. I don't think we need to rename
the metric because it's an internal diagnosis metric for our team.
R=fdoray@chromium.org, robliao@chromium.org
Bug: 756898, 810464
Change-Id: Ica379d6f3c21aebabeb0574847ba0bfcda346df8
Reviewed-on: https://chromium-review.googlesource.com/931524
Commit-Queue: Gabriel Charette <gab@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Cr-Commit-Position: refs/heads/master@{#538751}
CrOS-Libchrome-Original-Commit: a8863d2fc9bc0ede0e9f35306ac85cc0195f3e6c
2 files changed