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

Unified Diff: chrome/renderer/extensions/safe_builtins_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: chrome/renderer/extensions/safe_builtins_unittest.cc
diff --git a/chrome/renderer/extensions/safe_builtins_unittest.cc b/chrome/renderer/extensions/safe_builtins_unittest.cc
deleted file mode 100644
index 5b4f826fa02d174c656cf38757b4df84e1e8fb6f..0000000000000000000000000000000000000000
--- a/chrome/renderer/extensions/safe_builtins_unittest.cc
+++ /dev/null
@@ -1,67 +0,0 @@
-// Copyright 2013 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.
-
-#include "chrome/test/base/module_system_test.h"
-
-namespace extensions {
-namespace {
-
-class SafeBuiltinsUnittest : public ModuleSystemTest {
-};
-
-TEST_F(SafeBuiltinsUnittest, TestNotOriginalObject) {
- ModuleSystem::NativesEnabledScope natives_enabled_scope(
- env()->module_system());
- env()->RegisterModule("test",
- "var assert = requireNative('assert');\n"
- "Array.foo = 10;\n"
- "assert.AssertTrue(!$Array.hasOwnProperty('foo'));\n");
- env()->module_system()->Require("test");
-}
-
-TEST_F(SafeBuiltinsUnittest, TestSelf) {
- ModuleSystem::NativesEnabledScope natives_enabled_scope(
- env()->module_system());
- env()->RegisterModule("test",
- "var assert = requireNative('assert');\n"
- "Array.foo = 10;\n"
- "assert.AssertTrue($Array.self.foo == 10);\n"
- "var arr = $Array.self(1);\n"
- "assert.AssertTrue(arr.length == 1);\n"
- "assert.AssertTrue(arr[0] === undefined);\n");
- env()->module_system()->Require("test");
-}
-
-TEST_F(SafeBuiltinsUnittest, TestStaticFunction) {
- ModuleSystem::NativesEnabledScope natives_enabled_scope(
- env()->module_system());
- env()->RegisterModule("test",
- "var assert = requireNative('assert');\n"
- "Object.keys = function() {throw new Error()};\n"
- "var obj = {a: 10};\n"
- "var keys = $Object.keys(obj);\n"
- "assert.AssertTrue(keys.length == 1);\n"
- "assert.AssertTrue(keys[0] == 'a');\n");
- env()->module_system()->Require("test");
-}
-
-TEST_F(SafeBuiltinsUnittest, TestInstanceMethod) {
- ModuleSystem::NativesEnabledScope natives_enabled_scope(
- env()->module_system());
- env()->RegisterModule(
- "test",
- "var assert = requireNative('assert');\n"
- "Array.prototype.push = function() {throw new Error();}\n"
- "var arr = []\n"
- "$Array.push(arr, 1);\n"
- "assert.AssertTrue(arr.length == 1);\n"
- "assert.AssertTrue(arr[0] == 1);\n");
- env()->module_system()->Require("test");
-}
-
-// NOTE: JSON is already tested in ExtensionApiTest.Messaging, via
-// chrome/test/data/extensions/api_test/messaging/connect/page.js.
-
-} // namespace
-} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698