[Extensions] Convert more callers of CallModuleMethod

extensions/renderer/script_context.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_SCRIPT_CONTEXT_H_
 #define EXTENSIONS_RENDERER_SCRIPT_CONTEXT_H_
 
 #include <memory>
 #include <string>
 #include <utility>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/threading/thread_checker.h"
 #include "extensions/common/features/feature.h"
 #include "extensions/common/permissions/api_permission_set.h"
 #include "extensions/renderer/module_system.h"
 #include "extensions/renderer/request_sender.h"
 #include "extensions/renderer/safe_builtins.h"
+#include "extensions/renderer/script_injection_callback.h"
 #include "gin/runner.h"
 #include "url/gurl.h"
 #include "v8/include/v8.h"
 
 namespace blink {
 class WebFrame;
 class WebLocalFrame;
 }
 
 namespace content {
@@ skipping 78 matching lines @@
   content::RenderFrame* GetRenderFrame() const;
 
   // DEPRECATED.
   v8::Local<v8::Value> CallFunction(const v8::Local<v8::Function>& function,
                                     int argc,
                                     v8::Local<v8::Value> argv[]) const;
 
   // Safely calls the v8::Function, respecting the page load deferrer and
   // possibly executing asynchronously.
   // Doesn't catch exceptions; callers must do that if they want.
   // USE THIS METHOD RATHER THAN v8::Function::Call WHEREVER POSSIBLE.

[lazyboy, 2016/12/07 22:48:51]

THESE METHODS

   // TODO(devlin): Remove the above variants in favor of this.
   void SafeCallFunction(const v8::Local<v8::Function>& function,
                         int argc,
                         v8::Local<v8::Value> argv[]);
+  void SafeCallFunction(
+      const v8::Local<v8::Function>& function,
+      int argc,
+      v8::Local<v8::Value> argv[],
+      const ScriptInjectionCallback::CompleteCallback& callback);
 
   void DispatchEvent(const char* event_name, v8::Local<v8::Array> args) const;
 
   // Returns the availability of the API |api_name|.
   Feature::Availability GetAvailability(const std::string& api_name);
   // Returns the availability of the API |api_name|.
   // |check_alias| Whether API that has an alias that is available should be
   // considered available (even if the API itself is not available).
   Feature::Availability GetAvailability(const std::string& api_name,
                                         CheckAliasStatus check_alias);
@@ skipping 135 matching lines @@
   std::unique_ptr<Runner> runner_;
 
   base::ThreadChecker thread_checker_;
 
   DISALLOW_COPY_AND_ASSIGN(ScriptContext);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_RENDERER_SCRIPT_CONTEXT_H_