content: convert user action notification to observer usage

chrome/browser/ui/webui/user_actions/user_actions_ui_handler.h

Index: chrome/browser/ui/webui/user_actions/user_actions_ui_handler.h
diff --git a/chrome/browser/ui/webui/user_actions/user_actions_ui_handler.h b/chrome/browser/ui/webui/user_actions/user_actions_ui_handler.h
index de145ea6e5770272670a3e3063edcc2741639351..832ffeb060bc3d9b2b66874dbb30f85d6f79a003 100644
--- a/chrome/browser/ui/webui/user_actions/user_actions_ui_handler.h
+++ b/chrome/browser/ui/webui/user_actions/user_actions_ui_handler.h
@@ -5,33 +5,24 @@
 #ifndef CHROME_BROWSER_UI_WEBUI_USER_ACTIONS_USER_ACTIONS_UI_HANDLER_H_
 #define CHROME_BROWSER_UI_WEBUI_USER_ACTIONS_USER_ACTIONS_UI_HANDLER_H_
 
-#include "content/public/browser/notification_observer.h"
-#include "content/public/browser/notification_registrar.h"
+#include "content/public/browser/user_metrics.h"

[jam, 2013/01/30 17:53:24]

nit: doesn't look like this needs to be in the header?

 #include "content/public/browser/web_ui_message_handler.h"
 
 // UI Handler for chrome://user-actions/
 // It listens to user action notifications and passes those notifications
 // into the Javascript to update the page.
-class UserActionsUIHandler : public content::NotificationObserver,
-                             public content::WebUIMessageHandler {
+class UserActionsUIHandler : public content::WebUIMessageHandler {
  public:
   UserActionsUIHandler();
   virtual ~UserActionsUIHandler();
 
-  // NotificationObserver implementation:
-  // Listens for user action notifications and passes the message to
-  // observeUserAction in user_actions.js.
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) OVERRIDE;
+  void OnUserAction(const std::string& action);

[jam, 2013/01/30 17:53:24]

nit: this can be private, so hide it

[Paweł Hajdan Jr., 2013/01/31 14:34:44]

Done.

 
   // WebUIMessageHandler implementation:
   // Does nothing for now.
   virtual void RegisterMessages() OVERRIDE;
 
  private:
-  content::NotificationRegistrar registrar_;
-
   DISALLOW_COPY_AND_ASSIGN(UserActionsUIHandler);
 };