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

Unified Diff: chrome/renderer/extensions/utils_unittest.cc

Issue 359413004: Add support for using AMD modules from extensions modules. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: extensions_renderer should depend on gin Created 6 years, 5 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 | « chrome/renderer/extensions/safe_builtins_unittest.cc ('k') | chrome/test/DEPS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/extensions/utils_unittest.cc
diff --git a/chrome/renderer/extensions/utils_unittest.cc b/chrome/renderer/extensions/utils_unittest.cc
index 3a718ddccd14a4477469f25e7566a2054f5a8f7c..1763d86fbc254d1805b6bc10375bf29fb288c208 100644
--- a/chrome/renderer/extensions/utils_unittest.cc
+++ b/chrome/renderer/extensions/utils_unittest.cc
@@ -12,26 +12,26 @@ namespace {
class UtilsUnittest : public ModuleSystemTest {
protected:
void RegisterTestModule(const char* code) {
- RegisterModule("test",
- base::StringPrintf(
- "var assert = requireNative('assert');\n"
- "var AssertTrue = assert.AssertTrue;\n"
- "var AssertFalse = assert.AssertFalse;\n"
- "var utils = require('utils');\n"
- "%s",
- code));
+ env()->RegisterModule("test",
+ base::StringPrintf(
+ "var assert = requireNative('assert');\n"
+ "var AssertTrue = assert.AssertTrue;\n"
+ "var AssertFalse = assert.AssertFalse;\n"
+ "var utils = require('utils');\n"
+ "%s",
+ code));
}
private:
virtual void SetUp() OVERRIDE {
ModuleSystemTest::SetUp();
- RegisterModule("utils", IDR_UTILS_JS);
- OverrideNativeHandler("schema_registry",
- "exports.GetSchema = function() {};");
- OverrideNativeHandler("logging",
- "exports.CHECK = function() {};\n"
- "exports.WARNING = function() {};");
+ env()->RegisterModule("utils", IDR_UTILS_JS);
+ env()->OverrideNativeHandler("schema_registry",
+ "exports.GetSchema = function() {};");
+ env()->OverrideNativeHandler("logging",
+ "exports.CHECK = function() {};\n"
+ "exports.WARNING = function() {};");
}
};
@@ -41,7 +41,7 @@ TEST_F(UtilsUnittest, TestNothing) {
TEST_F(UtilsUnittest, SuperClass) {
ModuleSystem::NativesEnabledScope natives_enabled_scope(
- context_->module_system());
+ env()->module_system());
RegisterTestModule(
"function SuperClassImpl() {}\n"
"\n"
@@ -116,7 +116,7 @@ TEST_F(UtilsUnittest, SuperClass) {
"AssertTrue(subsub instanceof SubClass);\n"
"AssertTrue(subsub instanceof SubSubClass);\n");
- context_->module_system()->Require("test");
+ env()->module_system()->Require("test");
}
} // namespace
« no previous file with comments | « chrome/renderer/extensions/safe_builtins_unittest.cc ('k') | chrome/test/DEPS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698