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

Unified Diff: extensions/renderer/module_system_unittest.cc

Issue 375243002: Move ModuleSystemTest and the tests that use it into extensions/. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: namespace extensions 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
Index: extensions/renderer/module_system_unittest.cc
diff --git a/chrome/renderer/extensions/module_system_unittest.cc b/extensions/renderer/module_system_unittest.cc
similarity index 97%
rename from chrome/renderer/extensions/module_system_unittest.cc
rename to extensions/renderer/module_system_unittest.cc
index f99cb622256b6450e7dee2883764ae73b2595d7a..dafd7004acb7bedffc0b11170dae865c8b21b02c 100644
--- a/chrome/renderer/extensions/module_system_unittest.cc
+++ b/extensions/renderer/module_system_unittest.cc
@@ -3,23 +3,21 @@
// found in the LICENSE file.
#include "base/memory/scoped_ptr.h"
-#include "chrome/test/base/module_system_test.h"
#include "extensions/renderer/module_system.h"
+#include "extensions/renderer/module_system_test.h"
#include "gin/modules/module_registry.h"
-// TODO(cduvall/kalman): Put this file in extensions namespace.
-using extensions::ModuleSystem;
-using extensions::NativeHandler;
-using extensions::ObjectBackedNativeHandler;
+namespace extensions {
class CounterNatives : public ObjectBackedNativeHandler {
public:
- explicit CounterNatives(extensions::ChromeV8Context* context)
+ explicit CounterNatives(ScriptContext* context)
: ObjectBackedNativeHandler(context), counter_(0) {
- RouteFunction("Get", base::Bind(&CounterNatives::Get,
- base::Unretained(this)));
- RouteFunction("Increment", base::Bind(&CounterNatives::Increment,
- base::Unretained(this)));
+ RouteFunction("Get",
+ base::Bind(&CounterNatives::Get, base::Unretained(this)));
+ RouteFunction(
+ "Increment",
+ base::Bind(&CounterNatives::Increment, base::Unretained(this)));
}
void Get(const v8::FunctionCallbackInfo<v8::Value>& args) {
@@ -36,9 +34,7 @@ class CounterNatives : public ObjectBackedNativeHandler {
class TestExceptionHandler : public ModuleSystem::ExceptionHandler {
public:
- TestExceptionHandler()
- : handled_exception_(false) {
- }
+ TestExceptionHandler() : handled_exception_(false) {}
virtual void HandleUncaughtException(const v8::TryCatch& try_catch) OVERRIDE {
handled_exception_ = true;
@@ -487,3 +483,5 @@ TEST_F(ModuleSystemTest, TestRequireAsyncFromContextWithNoModuleSystem) {
env()->module_system()->Require("test");
RunResolvedPromises();
}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698