Commit 0d50a6c8 authored by Joshua Peraza's avatar Joshua Peraza Committed by Commit Bot

android: set custom browser ptype for browser reports

The custom browser ptype exists to differentiate single process webview
crashes from multi-process webview crashes, but was mistakenly set
for child process crashes instead of browser crashes.

Bug: 936243
Change-Id: I9e37f42323ae39a7a3fa8b81e71331c08aff35bb
Reviewed-on: https://chromium-review.googlesource.com/c/1490891Reviewed-by: default avatarMark Mentovai <mark@chromium.org>
Commit-Queue: Joshua Peraza <jperaza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#635843}
parent f2d25e31
......@@ -442,6 +442,11 @@ class HandlerStarter {
}
#if defined(OS_ANDROID)
std::string browser_ptype;
if (GetCrashReporterClient()->GetBrowserProcessType(&browser_ptype)) {
process_annotations["ptype"] = browser_ptype;
}
if (!base::PathExists(handler_path)) {
use_java_handler_ = true;
}
......@@ -487,13 +492,6 @@ class HandlerStarter {
BuildHandlerArgs(client, &database_path, &metrics_path, &url,
&process_annotations, &arguments);
#if defined(OS_ANDROID)
std::string browser_ptype;
if (GetCrashReporterClient()->GetBrowserProcessType(&browser_ptype)) {
process_annotations["ptype"] = browser_ptype;
}
#endif // defined(OS_ANDROID)
base::FilePath exe_dir;
base::FilePath handler_path;
if (!GetHandlerPath(&exe_dir, &handler_path)) {
......
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