aw: Fix multiprocess flag usage

components/crash/content/app/breakpad_linux.cc

Index: components/crash/content/app/breakpad_linux.cc
diff --git a/components/crash/content/app/breakpad_linux.cc b/components/crash/content/app/breakpad_linux.cc
index 036424c8368ce07c18eef7f3d03a676c1c78d8a5..0d3de65043a0b66fc1d3bc8fee578ae13179f611 100644
--- a/components/crash/content/app/breakpad_linux.cc
+++ b/components/crash/content/app/breakpad_linux.cc
@@ -1951,7 +1951,14 @@ void InitCrashReporter(const std::string& process_type) {
   if (parsed_command_line.HasSwitch(switches::kDisableBreakpad))
     return;
 
-  if (process_type.empty()) {
+  bool is_browser_process =
+#if defined(OS_ANDROID)
+      process_type == kWebViewSingleProcessType ||
+      process_type == kBrowserProcessType ||

[Torne, 2017/04/10 14:06:29]

How annoying would it be to change all the uses of empty string to "browser" so
this isn't OS-specific? (maybe not in this change since I assume we're going to
pick this to 58) Worth it?

[Tobias Sargeant, 2017/04/10 14:21:14]

It seems like it would be worth it. It's not our code though. I'm pretty sure
this CL removes the last webview use of "" as a process type.

I'll take a look with lower priority.

[Robert Sesek, 2017/04/10 15:17:18]

The process_type.empty() meaning browser is a baked-in assumption of Chrome,
since it stems from the fact that the browser process has no --type= switch.
Whether or not that's worth changing at the crash reporting level is debatable.

+#endif
+      process_type.empty();
+
+  if (is_browser_process) {
     bool enable_breakpad = GetCrashReporterClient()->GetCollectStatsConsent() ||
                            GetCrashReporterClient()->IsRunningUnattended();
     enable_breakpad &=