Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(84)

Side by Side Diff: extensions/renderer/module_system.h

Issue 2436893002: [Extensions] Convert some callers of ModuleSystem::CallModuleMethod (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « extensions/renderer/messaging_bindings.cc ('k') | extensions/renderer/module_system.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef EXTENSIONS_RENDERER_MODULE_SYSTEM_H_ 5 #ifndef EXTENSIONS_RENDERER_MODULE_SYSTEM_H_
6 #define EXTENSIONS_RENDERER_MODULE_SYSTEM_H_ 6 #define EXTENSIONS_RENDERER_MODULE_SYSTEM_H_
7 7
8 #include <map> 8 #include <map>
9 #include <memory> 9 #include <memory>
10 #include <set> 10 #include <set>
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 v8::Local<v8::Value> RunString(v8::Local<v8::String> code, 89 v8::Local<v8::Value> RunString(v8::Local<v8::String> code,
90 v8::Local<v8::String> name); 90 v8::Local<v8::String> name);
91 91
92 // Calls the specified method exported by the specified module. This is 92 // Calls the specified method exported by the specified module. This is
93 // equivalent to calling require('module_name').method_name() from JS. 93 // equivalent to calling require('module_name').method_name() from JS.
94 // DEPRECATED: see crbug.com/629431 94 // DEPRECATED: see crbug.com/629431
95 // TODO(devlin): Remove these. 95 // TODO(devlin): Remove these.
96 v8::Local<v8::Value> CallModuleMethod( 96 v8::Local<v8::Value> CallModuleMethod(
97 const std::string& module_name, 97 const std::string& module_name,
98 const std::string& method_name); 98 const std::string& method_name);
99 v8::Local<v8::Value> CallModuleMethod(
100 const std::string& module_name,
101 const std::string& method_name,
102 std::vector<v8::Local<v8::Value>>* args);
103 v8::Local<v8::Value> CallModuleMethod(const std::string& module_name, 99 v8::Local<v8::Value> CallModuleMethod(const std::string& module_name,
104 const std::string& method_name, 100 const std::string& method_name,
105 int argc, 101 int argc,
106 v8::Local<v8::Value> argv[]); 102 v8::Local<v8::Value> argv[]);
107 103
108 // Same as the above, but allows for blocking execution. 104 // Same as the above, but allows for blocking execution.
109 void CallModuleMethodSafe(const std::string& module_name, 105 void CallModuleMethodSafe(const std::string& module_name,
110 const std::string& method_name); 106 const std::string& method_name);
111 void CallModuleMethodSafe(const std::string& module_name, 107 void CallModuleMethodSafe(const std::string& module_name,
112 const std::string& method_name, 108 const std::string& method_name,
(...skipping 151 matching lines...) Expand 10 before | Expand all | Expand 10 after
264 std::vector<std::unique_ptr<NativeHandler>> clobbered_native_handlers_; 260 std::vector<std::unique_ptr<NativeHandler>> clobbered_native_handlers_;
265 261
266 base::WeakPtrFactory<ModuleSystem> weak_factory_; 262 base::WeakPtrFactory<ModuleSystem> weak_factory_;
267 263
268 DISALLOW_COPY_AND_ASSIGN(ModuleSystem); 264 DISALLOW_COPY_AND_ASSIGN(ModuleSystem);
269 }; 265 };
270 266
271 } // namespace extensions 267 } // namespace extensions
272 268
273 #endif // EXTENSIONS_RENDERER_MODULE_SYSTEM_H_ 269 #endif // EXTENSIONS_RENDERER_MODULE_SYSTEM_H_
OLDNEW
« no previous file with comments | « extensions/renderer/messaging_bindings.cc ('k') | extensions/renderer/module_system.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698