GitLab

ProcessMetrics silently ignores failures in GetProcessTimes because of
concerns about race conditions, but since ProcessMetrics *owns* a
duplicated process handle such concerns are not valid. If they were then
we could get occasional handle errors when debugging or using
Application Verifier which is unacceptable any. So, adding a DCHECK.

Change-Id: I97475f4223ab71be87ae9025d3f8a00fe2363ff0
Bug: 890051
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1601540Reviewed-by: David Bienvenu <davidbienvenu@chromium.org>
Reviewed-by: Bruce Dawson <brucedawson@chromium.org>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#658393}