Make ExtensionMsg_MessageInvoke run a module system function rather than a

chrome/renderer/extensions/extension_helper.cc

Index: chrome/renderer/extensions/extension_helper.cc
diff --git a/chrome/renderer/extensions/extension_helper.cc b/chrome/renderer/extensions/extension_helper.cc
index a5e7ef48347afc013c0d161442927ac040bf6f32..2d9bb711169bbf753d2083b7a7d15ad98449bd02 100644
--- a/chrome/renderer/extensions/extension_helper.cc
+++ b/chrome/renderer/extensions/extension_helper.cc
@@ -11,6 +11,7 @@
 #include "base/lazy_instance.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
+#include "base/values.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/extension_messages.h"
 #include "chrome/common/render_messages.h"
@@ -250,16 +251,20 @@ void ExtensionHelper::OnExtensionResponse(int request_id,
 }
 
 void ExtensionHelper::OnExtensionMessageInvoke(const std::string& extension_id,
+                                               const std::string& module_name,
                                                const std::string& function_name,
                                                const base::ListValue& args,
                                                bool user_gesture) {
   scoped_ptr<WebScopedUserGesture> web_user_gesture;
-  if (user_gesture) {
+  if (user_gesture)
     web_user_gesture.reset(new WebScopedUserGesture);
-  }
-
-  dispatcher_->v8_context_set().DispatchChromeHiddenMethod(
-      extension_id, function_name, args, render_view());
+  dispatcher_->v8_context_set().ForEach(

[koz (OOO until 15th September), 2013/05/31 04:28:44]

It feels a little weird to have to pass Dispatcher::CallModuleMethod like this.
Consider refactoring to look like

dispatcher_->v8_context_set()->CallModuleMethodOnAllContexts(...);

Or even

dispatcher_->CallModuleMethodOnAllContexts(...);

This would clean up a bunch of other ForEach() call sites, too. Up to you,
though.

[not at google - send to devlin, 2013/05/31 22:47:07]

I had decided to do it this way because there were the 3 different forms of
CallModuleMethod and it seems an arbitrary choice which of the forms to lift
onto ChromeV8Context - certainly I wouldn't lift a ListValue version onto it,
which is what all of these take.

However I agree it's weird. I'll factor this a bit differently and share the
concept at a higher level.

+      extension_id,
+      render_view(),
+      base::Bind(&Dispatcher::CallModuleMethod,
+                 module_name,
+                 function_name,
+                 &args));
 }
 
 void ExtensionHelper::OnExtensionDispatchOnConnect(
@@ -364,7 +369,8 @@ void ExtensionHelper::OnAppWindowClosed() {
       dispatcher_->v8_context_set().GetByV8Context(script_context);
   if (!chrome_v8_context)
     return;
-  chrome_v8_context->CallChromeHiddenMethod("OnAppWindowClosed", 0, NULL, NULL);
+  chrome_v8_context->module_system()->CallModuleMethod(
+      "app.window", "onAppWindowClosed");
 }
 
 }  // namespace extensions