crash-reporter: Send systemwide stability information up to Chrome

crash_reporter.cc

 // Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <string>
 
 #include "base/file_util.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/string_util.h"
@@ skipping 35 matching lines @@
 
 static bool IsFeedbackAllowed() {
   return s_metrics_lib.AreMetricsEnabled();
 }
 
 static bool TouchFile(const FilePath &file_path) {
   return file_util::WriteFile(file_path, "", 0) == 0;
 }
 
 static void CountKernelCrash() {
+  // TODO(kmixter): We can remove this histogram once we have

[petkov, 2011/01/10 18:51:41]

File a cleanup issue?

+  // UMA stability data.
   s_metrics_lib.SendEnumToUMA(std::string(kCrashCounterHistogram),
                               kCrashKindKernel,
                               kCrashKindMax);
+  s_metrics_lib.SendCrashToUMA("kernel");
 }
 
 static void CountUncleanShutdown() {
+  // TODO(kmixter): We can remove this histogram once we have
+  // UMA stability data.
   s_metrics_lib.SendEnumToUMA(std::string(kCrashCounterHistogram),
                               kCrashKindUncleanShutdown,
                               kCrashKindMax);
+  s_metrics_lib.SendCrashToUMA("uncleanshutdown");
 }
 
 static void CountUserCrash() {
+  // TODO(kmixter): We can remove this histogram once we have
+  // UMA stability data, as well as the signalling to
+  // metrics daemon.
   s_metrics_lib.SendEnumToUMA(std::string(kCrashCounterHistogram),
                               kCrashKindUser,
                               kCrashKindMax);
+  s_metrics_lib.SendCrashToUMA("user");
   std::string command = StringPrintf(
       "/usr/bin/dbus-send --type=signal --system / \"%s\" &",
       kUserCrashSignal);
   // Announce through D-Bus whenever a user crash happens. This is
   // used by the metrics daemon to log active use time between
   // crashes.
   //
   // This could be done more efficiently by explicit fork/exec or
   // using a dbus library directly. However, this should run
   // relatively rarely and longer term we may need to implement a
@@ skipping 124 matching lines @@
     user_collector.Disable();
     return 0;
   }
 
   if (!FLAGS_generate_kernel_signature.empty()) {
     return GenerateKernelSignature(&kernel_collector);
   }
 
   return HandleUserCrash(&user_collector);
 }