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

Unified Diff: extensions/renderer/utils_unittest.cc

Issue 2423043003: [Extensions] Convert some callers of ScriptContext::CallFunction (Closed)
Patch Set: d'oh 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « extensions/renderer/module_system.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/renderer/utils_unittest.cc
diff --git a/extensions/renderer/utils_unittest.cc b/extensions/renderer/utils_unittest.cc
index 485c214ea66feab21cd8157b82cfa31be2d10f98..b45c35931f00e8910b3e054676a80bd88d257c33 100644
--- a/extensions/renderer/utils_unittest.cc
+++ b/extensions/renderer/utils_unittest.cc
@@ -38,38 +38,39 @@ TEST_F(UtilsUnittest, TestNothing) {
TEST_F(UtilsUnittest, SuperClass) {
ModuleSystem::NativesEnabledScope natives_enabled_scope(
env()->module_system());
- env()->module_system()->CallModuleMethod("utils_unittest", "testSuperClass");
+ env()->module_system()->CallModuleMethodSafe("utils_unittest",
+ "testSuperClass");
}
TEST_F(UtilsUnittest, PromiseNoResult) {
ModuleSystem::NativesEnabledScope natives_enabled_scope(
env()->module_system());
- env()->module_system()->CallModuleMethod("utils_unittest",
- "testPromiseNoResult");
+ env()->module_system()->CallModuleMethodSafe("utils_unittest",
+ "testPromiseNoResult");
RunResolvedPromises();
}
TEST_F(UtilsUnittest, PromiseOneResult) {
ModuleSystem::NativesEnabledScope natives_enabled_scope(
env()->module_system());
- env()->module_system()->CallModuleMethod("utils_unittest",
- "testPromiseOneResult");
+ env()->module_system()->CallModuleMethodSafe("utils_unittest",
+ "testPromiseOneResult");
RunResolvedPromises();
}
TEST_F(UtilsUnittest, PromiseTwoResults) {
ModuleSystem::NativesEnabledScope natives_enabled_scope(
env()->module_system());
- env()->module_system()->CallModuleMethod("utils_unittest",
- "testPromiseTwoResults");
+ env()->module_system()->CallModuleMethodSafe("utils_unittest",
+ "testPromiseTwoResults");
RunResolvedPromises();
}
TEST_F(UtilsUnittest, PromiseError) {
ModuleSystem::NativesEnabledScope natives_enabled_scope(
env()->module_system());
- env()->module_system()->CallModuleMethod("utils_unittest",
- "testPromiseError");
+ env()->module_system()->CallModuleMethodSafe("utils_unittest",
+ "testPromiseError");
RunResolvedPromises();
}
« no previous file with comments | « extensions/renderer/module_system.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698