crash-reporter: Ignore Chrome crashes, relying on Chrome breakpad

user_collector.cc

Index: user_collector.cc
diff --git a/user_collector.cc b/user_collector.cc
index 02025c20d9db86a53b3193e4c5474721a42cd1e1..7a1a7cb5cbbd977cc48e6ef63780453b6b7f9f09 100644
--- a/user_collector.cc
+++ b/user_collector.cc
@@ -441,9 +441,21 @@ bool UserCollector::HandleCrash(int signal, int pid, const char *force_exec) {
   }
 
   bool feedback = is_feedback_allowed_function_();
+  const char *handling_string = "handling";
+  if (!feedback) {
+    handling_string = "ignoring - no consent";
+  }
+
+  // Treat Chrome crashes as if the user opted-out.  We stop counting Chrome
+  // crashes towards user crashes, so user crashes really mean non-Chrome
+  // user-space crashes.
+  if (exec == "chrome") {
+    feedback = false;

[petkov, 2010/11/09 23:30:12]

probably doesn't matter but feedback may be false already. not sure if you care
to mark these as ignoring because it's chrome or ignoring because there's no
consent. or maybe you want a separate string.

+    handling_string = "ignoring - chrome crash";
+  }
+
   logger_->LogWarning("Received crash notification for %s[%d] sig %d (%s)",
-                      exec.c_str(), pid, signal,
-                      feedback ? "handling" : "ignoring - no consent");
+                      exec.c_str(), pid, signal, handling_string);
 
   if (feedback) {
     count_crash_function_();