[Extensions] Remove unsafe variants of CallModuleMethod

extensions/renderer/module_system.h

 // 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.
 
 #ifndef EXTENSIONS_RENDERER_MODULE_SYSTEM_H_
 #define EXTENSIONS_RENDERER_MODULE_SYSTEM_H_
 
 #include <map>
 #include <memory>
 #include <set>
@@ skipping 66 matching lines @@
   // require('module_name') from the loaded JS files.
   v8::MaybeLocal<v8::Object> Require(const std::string& module_name);
   void Require(const v8::FunctionCallbackInfo<v8::Value>& args);
 
   // Run |code| in the current context with the name |name| used for stack
   // traces.
   v8::Local<v8::Value> RunString(v8::Local<v8::String> code,
                                  v8::Local<v8::String> name);
 
   // Calls the specified method exported by the specified module. This is
-  // equivalent to calling require('module_name').method_name() from JS.
-  // DEPRECATED: see crbug.com/629431
-  // TODO(devlin): Remove these.
-  v8::Local<v8::Value> CallModuleMethod(const std::string& module_name,
-                                        const std::string& method_name,
-                                        int argc,
-                                        v8::Local<v8::Value> argv[]);
-
-  // Same as the above, but allows for blocking execution.
+  // equivalent to calling require('module_name').method_name() from JS. Note:
+  // this may result in asynchronous execution if javascript is presently
+  // disabled.
+  // TODO(devlin): Rename this to just CallModuleMethod()?
   void CallModuleMethodSafe(const std::string& module_name,
                             const std::string& method_name);
   void CallModuleMethodSafe(const std::string& module_name,
                             const std::string& method_name,
                             std::vector<v8::Local<v8::Value>>* args);
   void CallModuleMethodSafe(const std::string& module_name,
                             const std::string& method_name,
                             int argc,
                             v8::Local<v8::Value> argv[]);
   void CallModuleMethodSafe(
@@ skipping 175 matching lines @@
   std::set<std::string> loaded_modules_;
 
   base::WeakPtrFactory<ModuleSystem> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ModuleSystem);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_RENDERER_MODULE_SYSTEM_H_