CPM Refactor and Cleanup

chrome/browser/performance_monitor/performance_monitor.h

Index: chrome/browser/performance_monitor/performance_monitor.h
diff --git a/chrome/browser/performance_monitor/performance_monitor.h b/chrome/browser/performance_monitor/performance_monitor.h
index c74ee7ae6921319716efbbd2e195f44cbdfcee94..e7d7b29f98ce3ea64dbcdae1c1b2c26eac7926c6 100644
--- a/chrome/browser/performance_monitor/performance_monitor.h
+++ b/chrome/browser/performance_monitor/performance_monitor.h
@@ -18,6 +18,11 @@
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/notification_source.h"
+#include "content/public/browser/render_process_host.h"
+
+namespace extensions {
+class Extension;
+}
 
 namespace performance_monitor {
 class Database;
@@ -107,6 +112,16 @@ class PerformanceMonitor : public content::NotificationObserver {
   // Perform any collections that are done on a timed basis.
   void DoTimedCollections();
 
+  // Generate an appropriate ExtensionEvent for an extension-related occurrance
+  // and insert it in the database.
+  void HandleExtensionEvent(EventType type,

[Yoyo Zhou, 2012/08/02 08:39:01]

Why "Handle" and not "Add" for these names?

+                            const extensions::Extension* extension);
+
+  // Generate an appropriate CrashEvent for a renderer crash and insert it in
+  // the database.
+  void HandleCrashEvent(
+      const content::RenderProcessHost::RendererClosedDetails& details);
+
   // The location at which the database files are stored; if empty, the database
   // will default to '<user_data_dir>/performance_monitor_dbs'.
   FilePath database_path_;