Remove deprecated extension EventRouter APIs.

chrome/browser/extensions/menu_manager_unittest.cc

Index: chrome/browser/extensions/menu_manager_unittest.cc
diff --git a/chrome/browser/extensions/menu_manager_unittest.cc b/chrome/browser/extensions/menu_manager_unittest.cc
index c707144482e6f5e23e65304a5832e33ec37a5eaa..b8a368d93de52459776e3bd1bc8fd8142b436f3b 100644
--- a/chrome/browser/extensions/menu_manager_unittest.cc
+++ b/chrome/browser/extensions/menu_manager_unittest.cc
@@ -457,13 +457,13 @@ class MockEventRouter : public EventRouter {
                     EventRouter::UserGestureState state));
 
   virtual void DispatchEventToExtension(const std::string& extension_id,
-                                        const std::string& event_name,
-                                        scoped_ptr<base::ListValue> event_args,
-                                        Profile* source_profile,
-                                        const GURL& event_url,
-                                        EventRouter::UserGestureState state) {
-    DispatchEventToExtensionMock(extension_id, event_name, event_args.release(),
-                                 source_profile, event_url, state);
+                                        scoped_ptr<Event> event) {
+    DispatchEventToExtensionMock(extension_id,

[Yoyo Zhou, 2012/12/05 22:28:32]

This mock method doesn't make sense anymore; it should reflect the new
interface. Don't know if you want to postpone fixing that to a later CL.

[Matt Perry, 2012/12/05 22:53:53]

If you look later in the file, it uses gmock to check that the method is called
with certain parameters. I don't know if it's possible to do that if you change
the mock to accept a single Event struct.

+                                 event->event_name,
+                                 event->event_args.release(),
+                                 event->restrict_to_profile,
+                                 event->event_url,
+                                 event->user_gesture);
   }
 
  private: