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

Unified Diff: chrome/browser/extensions/api/content_settings/content_settings_store_unittest.cc

Issue 10381097: Move content settings extension API to content_settings dir. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: fixed conflict Created 8 years, 7 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/browser/extensions/api/content_settings/content_settings_store_unittest.cc
diff --git a/chrome/browser/extensions/extension_content_settings_store_unittest.cc b/chrome/browser/extensions/api/content_settings/content_settings_store_unittest.cc
similarity index 91%
rename from chrome/browser/extensions/extension_content_settings_store_unittest.cc
rename to chrome/browser/extensions/api/content_settings/content_settings_store_unittest.cc
index ccbc0e2e4efc087d4001df04e80df808b2c638d5..c067cf769226c8935b211e43d1d706311c27208c 100644
--- a/chrome/browser/extensions/extension_content_settings_store_unittest.cc
+++ b/chrome/browser/extensions/api/content_settings/content_settings_store_unittest.cc
@@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/extensions/extension_content_settings_store.h"
+#include "chrome/browser/extensions/api/content_settings/content_settings_store.h"
#include "base/memory/scoped_ptr.h"
#include "chrome/browser/content_settings/content_settings_rule.h"
@@ -12,6 +12,8 @@
using ::testing::Mock;
+namespace extensions {
+
namespace {
void CheckRule(const content_settings::Rule& rule,
@@ -36,15 +38,15 @@ class FakeTimer {
int64 internal_;
};
-class MockExtensionContentSettingsStoreObserver
- : public ExtensionContentSettingsStore::Observer {
+class MockContentSettingsStoreObserver
+ : public ContentSettingsStore::Observer {
public:
MOCK_METHOD2(OnContentSettingChanged,
void(const std::string& extension_id, bool incognito));
};
ContentSetting GetContentSettingFromStore(
- const ExtensionContentSettingsStore* store,
+ const ContentSettingsStore* store,
const GURL& primary_url, const GURL& secondary_url,
ContentSettingsType content_type,
const std::string& resource_identifier,
@@ -58,7 +60,7 @@ ContentSetting GetContentSettingFromStore(
}
void GetSettingsForOneTypeFromStore(
- const ExtensionContentSettingsStore* store,
+ const ContentSettingsStore* store,
ContentSettingsType content_type,
const std::string& resource_identifier,
bool incognito,
@@ -72,10 +74,10 @@ void GetSettingsForOneTypeFromStore(
} // namespace
-class ExtensionContentSettingsStoreTest : public ::testing::Test {
+class ContentSettingsStoreTest : public ::testing::Test {
public:
- ExtensionContentSettingsStoreTest() :
- store_(new ExtensionContentSettingsStore()) {
+ ContentSettingsStoreTest() :
+ store_(new ContentSettingsStore()) {
}
protected:
@@ -83,17 +85,17 @@ class ExtensionContentSettingsStoreTest : public ::testing::Test {
store_->RegisterExtension(ext_id, timer_.GetNext(), true);
}
- ExtensionContentSettingsStore* store() {
+ ContentSettingsStore* store() {
return store_.get();
}
private:
FakeTimer timer_;
- scoped_refptr<ExtensionContentSettingsStore> store_;
+ scoped_refptr<ContentSettingsStore> store_;
};
-TEST_F(ExtensionContentSettingsStoreTest, RegisterUnregister) {
- ::testing::StrictMock<MockExtensionContentSettingsStoreObserver> observer;
+TEST_F(ContentSettingsStoreTest, RegisterUnregister) {
+ ::testing::StrictMock<MockContentSettingsStoreObserver> observer;
store()->AddObserver(&observer);
GURL url("http://www.youtube.com");
@@ -184,7 +186,7 @@ TEST_F(ExtensionContentSettingsStoreTest, RegisterUnregister) {
store()->RemoveObserver(&observer);
}
-TEST_F(ExtensionContentSettingsStoreTest, GetAllSettings) {
+TEST_F(ContentSettingsStoreTest, GetAllSettings) {
bool incognito = false;
std::vector<content_settings::Rule> rules;
GetSettingsForOneTypeFromStore(
@@ -249,3 +251,5 @@ TEST_F(ExtensionContentSettingsStoreTest, GetAllSettings) {
store(), CONTENT_SETTINGS_TYPE_COOKIES, "", incognito, &rules);
ASSERT_EQ(0u, rules.size());
}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698