Crash reporter: collect suspend and resume info from power manager

unclean_shutdown_collector.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 "crash-reporter/unclean_shutdown_collector.h"
 
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "crash-reporter/system_logging.h"
 
 static const char kUncleanShutdownFile[] =
     "/var/lib/crash_reporter/pending_clean_shutdown";
 
+// Files created by power manager used for crash reporting.
+static const char kPowerdTracePath[] = "/var/lib/power_manager";
+// File with timestamp of last suspend.
+static const char kPowerdSuspended[] = "powerd_suspended";
+// Presence of this file indicates that the battery is critically low.
+static const char kPowerdLowBattery[] = "powerd_low_battery";
+
 UncleanShutdownCollector::UncleanShutdownCollector()
-    : unclean_shutdown_file_(kUncleanShutdownFile) {
+    : unclean_shutdown_file_(kUncleanShutdownFile),
+      powerd_trace_path_(kPowerdTracePath),
+      powerd_suspend_file_(powerd_trace_path_.Append(kPowerdSuspended)),
+      powerd_low_battery_file_(powerd_trace_path_.Append(kPowerdLowBattery)) {
 }
 
 UncleanShutdownCollector::~UncleanShutdownCollector() {
 }
 
 bool UncleanShutdownCollector::Enable() {
   FilePath file_path(unclean_shutdown_file_);
   file_util::CreateDirectory(file_path.DirName());
   if (file_util::WriteFile(file_path, "", 0) != 0) {
     logger_->LogError("Unable to create shutdown check file");
@@ skipping 12 matching lines @@
 }
 
 bool UncleanShutdownCollector::Collect() {
   FilePath unclean_file_path(unclean_shutdown_file_);
   if (!file_util::PathExists(unclean_file_path)) {
     return false;
   }
   logger_->LogWarning("Last shutdown was not clean");
   DeleteUncleanShutdownFile();
 
+  CheckForDeadBatteryUncleanShutdown();
+  file_util::Delete(powerd_suspend_file_, false);

[kmixter1, 2010/11/19 23:43:27]

Still would be nice to factor this out.  How about putting it into
DeleteUncleanShutdownFiles? and calling that above
CheckForDeadBatteryUncleanShutdown?  Then removing these deletions in Disable.

+  file_util::Delete(powerd_low_battery_file_, false);
+
   if (is_feedback_allowed_function_()) {
     count_crash_function_();
   }
   return true;
 }
 
 bool UncleanShutdownCollector::Disable() {
   logger_->LogInfo("Clean shutdown signalled");
+  file_util::Delete(powerd_suspend_file_, false);
+  file_util::Delete(powerd_low_battery_file_, false);
   return DeleteUncleanShutdownFile();
 }
+
+bool UncleanShutdownCollector::CheckForDeadBatteryUncleanShutdown()
+{
+  // Check for case of battery running out while suspended.
+  if (file_util::PathExists(powerd_suspend_file_)) {
+    logger_->LogInfo("Unclean shutdown occurred while suspended. The battery "
+                     "probably ran out.");
+    return true;
+  }
+  // Check for case of battery running out after resuming from a low-battery
+  // suspend.
+  if (file_util::PathExists(powerd_low_battery_file_)) {
+    logger_->LogInfo("Unclean shutdown occurred while resumed after battery "

[kmixter1, 2010/11/19 23:43:27]

how about "while running with critically low battery" since there wasn't
necessarily a suspend/resume cycle.  Instead of "The battery probably ran out"
here and above how about "not counting towards unclean shutdown statistic."  (or
something to that effect)

+                     "was critically low.  The battery probably ran out.");
+    return true;
+  }
+  return false;
+}