Commit d5ad9c4b authored by Peter Wen's avatar Peter Wen Committed by Commit Bot

Android: Fix stability proto renderer rate

Only increment stability proto renderer crash rate if the crash dump is
valid.

BUG=786992

Change-Id: I63e4d24bdd9d258775bba21b611414770424d57c
Reviewed-on: https://chromium-review.googlesource.com/779480
Commit-Queue: Peter Wen <wnwen@chromium.org>
Reviewed-by: default avatarAlexei Svitkine <asvitkine@chromium.org>
Reviewed-by: default avatarYaron Friedman <yfriedman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#517890}
parent 39a3080e
......@@ -143,7 +143,9 @@ void ChromeStabilityMetricsProvider::OnCrashDumpProcessed(
const breakpad::CrashDumpManager::CrashDumpDetails& details) {
// There is a delay for OOM flag to be removed when app goes to background, so
// we can't just check for OOM_PROTECTED flag.
if (details.process_type == content::PROCESS_TYPE_RENDERER &&
if (details.status ==
breakpad::CrashDumpManager::CrashDumpStatus::kValidDump &&
details.process_type == content::PROCESS_TYPE_RENDERER &&
details.termination_status == base::TERMINATION_STATUS_OOM_PROTECTED &&
(details.app_state ==
base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES ||
......
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