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

Unified Diff: chrome/browser/extensions/api/api_resource_manager_unittest.cc

Issue 319543006: Move c/b/e/api/api_resource_manager_unittest.cc to (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use TestBrowserThreadBundle instead Created 6 years, 6 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 | « no previous file | chrome/browser/extensions/api/web_request/web_request_api.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/api_resource_manager_unittest.cc
diff --git a/chrome/browser/extensions/api/api_resource_manager_unittest.cc b/chrome/browser/extensions/api/api_resource_manager_unittest.cc
deleted file mode 100644
index 865c4afeb7b0569b34182c0422e14ecba7d89f38..0000000000000000000000000000000000000000
--- a/chrome/browser/extensions/api/api_resource_manager_unittest.cc
+++ /dev/null
@@ -1,66 +0,0 @@
-// Copyright (c) 2012 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 "base/files/file_path.h"
-#include "base/strings/string_util.h"
-#include "chrome/browser/extensions/extension_function_test_utils.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/common/extensions/extension_test_util.h"
-#include "chrome/test/base/browser_with_test_window_test.h"
-#include "content/public/browser/browser_thread.h"
-#include "extensions/browser/api/api_resource.h"
-#include "extensions/browser/api/api_resource_manager.h"
-#include "extensions/common/extension.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "url/gurl.h"
-
-namespace utils = extension_function_test_utils;
-
-using content::BrowserThread;
-
-namespace extensions {
-
-// TODO(rockot): Move these tests to src/extensions.
-class ApiResourceManagerUnitTest : public BrowserWithTestWindowTest {
- public:
- virtual void SetUp() { BrowserWithTestWindowTest::SetUp(); }
-};
-
-class FakeApiResource : public ApiResource {
- public:
- explicit FakeApiResource(const std::string& owner_extension_id)
- : ApiResource(owner_extension_id) {}
- virtual ~FakeApiResource() {}
- static const BrowserThread::ID kThreadId = BrowserThread::UI;
-};
-
-TEST_F(ApiResourceManagerUnitTest, TwoAppsCannotShareResources) {
- scoped_ptr<ApiResourceManager<FakeApiResource> > manager(
- new ApiResourceManager<FakeApiResource>(NULL));
- scoped_refptr<extensions::Extension> extension_one(
- utils::CreateEmptyExtension("one"));
- scoped_refptr<extensions::Extension> extension_two(
- utils::CreateEmptyExtension("two"));
-
- const std::string extension_one_id(extension_one->id());
- const std::string extension_two_id(extension_two->id());
-
- int resource_one_id = manager->Add(new FakeApiResource(extension_one_id));
- int resource_two_id = manager->Add(new FakeApiResource(extension_two_id));
- CHECK(resource_one_id);
- CHECK(resource_two_id);
-
- // Confirm each extension can get its own resource.
- ASSERT_TRUE(manager->Get(extension_one_id, resource_one_id) != NULL);
- ASSERT_TRUE(manager->Get(extension_two_id, resource_two_id) != NULL);
-
- // Confirm neither extension can get the other's resource.
- ASSERT_TRUE(manager->Get(extension_one_id, resource_two_id) == NULL);
- ASSERT_TRUE(manager->Get(extension_two_id, resource_one_id) == NULL);
-
- // And make sure we're not susceptible to any Jedi mind tricks.
- ASSERT_TRUE(manager->Get(std::string(), resource_one_id) == NULL);
-}
-
-} // namespace extensions
« no previous file with comments | « no previous file | chrome/browser/extensions/api/web_request/web_request_api.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698