[Extensions] Convert some callers of ModuleSystem::CallModuleMethod

extensions/renderer/messaging_bindings.cc

 // Copyright 2014 The Chromium 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 "extensions/renderer/messaging_bindings.h"
 
 #include <stdint.h>
 
 #include <map>
 #include <string>
@@ skipping 195 matching lines @@
   if (message.user_gesture) {
     web_user_gesture.reset(new blink::WebScopedUserGesture);
 
     if (script_context->web_frame()) {
       blink::WebDocument document = script_context->web_frame()->document();
       allow_window_focus.reset(new blink::WebScopedWindowFocusAllowedIndicator(
           &document));
     }
   }
 
-  script_context->module_system()->CallModuleMethod(
+  script_context->module_system()->CallModuleMethodSafe(
       "messaging", "dispatchOnMessage", &arguments);
 }
 
 void DispatchOnDisconnectToScriptContext(int global_port_id,
                                          const std::string& error_message,
                                          ScriptContext* script_context) {
   MessagingBindings* bindings = g_messaging_map.Get()[script_context];
   DCHECK(bindings);
   ExtensionPort* port = bindings->GetPortWithGlobalId(global_port_id);
   if (!port)
     return;
 
   v8::Isolate* isolate = script_context->isolate();
   v8::HandleScope handle_scope(isolate);
 
   std::vector<v8::Local<v8::Value>> arguments;
   arguments.push_back(v8::Integer::New(isolate, port->local_id()));
   v8::Local<v8::String> v8_error_message;
   if (!error_message.empty())
     ToV8String(isolate, error_message.c_str(), &v8_error_message);
   if (!v8_error_message.IsEmpty()) {
     arguments.push_back(v8_error_message);
   } else {
     arguments.push_back(v8::Null(isolate));
   }
 
-  script_context->module_system()->CallModuleMethod(
+  script_context->module_system()->CallModuleMethodSafe(
       "messaging", "dispatchOnDisconnect", &arguments);
 }
 
 }  // namespace
 
 MessagingBindings::MessagingBindings(ScriptContext* context)
     : ObjectBackedNativeHandler(context), weak_ptr_factory_(this) {
   g_messaging_map.Get()[context] = this;
   RouteFunction("CloseChannel", base::Bind(&MessagingBindings::CloseChannel,
                                            base::Unretained(this)));
@@ skipping 299 matching lines @@
   DCHECK(iter != disconnected_ports_.end());
   iter->second->SetGlobalId(global_id);
   // Setting the global id dispatches pending messages, so we can delete the
   // port now.
   disconnected_ports_.erase(iter);
 }
 
 int MessagingBindings::GetNextLocalId() { return next_local_id_++; }
 
 }  // namespace extensions