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

Unified Diff: chrome/browser/content_settings/content_settings_internal_extension_provider.cc

Issue 10823370: rename ExtensionProvider and PlatformAppProvider class (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 8 years, 4 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/content_settings/content_settings_internal_extension_provider.cc
diff --git a/chrome/browser/content_settings/content_settings_platform_app_provider.cc b/chrome/browser/content_settings/content_settings_internal_extension_provider.cc
similarity index 88%
rename from chrome/browser/content_settings/content_settings_platform_app_provider.cc
rename to chrome/browser/content_settings/content_settings_internal_extension_provider.cc
index 18a2e88c8039d3748a758140d90938325bc091e9..5457254eddb28b650f5add32886bcb42fe0d98a5 100644
--- a/chrome/browser/content_settings/content_settings_platform_app_provider.cc
+++ b/chrome/browser/content_settings/content_settings_internal_extension_provider.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/content_settings/content_settings_platform_app_provider.h"
+#include "chrome/browser/content_settings/content_settings_internal_extension_provider.h"
#include "chrome/browser/content_settings/content_settings_rule.h"
#include "chrome/browser/extensions/extension_host.h"
@@ -21,7 +21,8 @@ using extensions::UnloadedExtensionInfo;
namespace content_settings {
-PlatformAppProvider::PlatformAppProvider(ExtensionService* extension_service)
+InternalExtensionProvider::InternalExtensionProvider(
+ ExtensionService* extension_service)
: registrar_(new content::NotificationRegistrar) {
// Whitelist all extensions loaded so far.
const ExtensionSet* extensions = extension_service->extensions();
@@ -39,18 +40,18 @@ PlatformAppProvider::PlatformAppProvider(ExtensionService* extension_service)
content::Source<Profile>(profile));
}
-PlatformAppProvider::~PlatformAppProvider() {
+InternalExtensionProvider::~InternalExtensionProvider() {
DCHECK(!registrar_.get());
}
-RuleIterator* PlatformAppProvider::GetRuleIterator(
+RuleIterator* InternalExtensionProvider::GetRuleIterator(
ContentSettingsType content_type,
const ResourceIdentifier& resource_identifier,
bool incognito) const {
return value_map_.GetRuleIterator(content_type, resource_identifier, &lock_);
}
-bool PlatformAppProvider::SetWebsiteSetting(
+bool InternalExtensionProvider::SetWebsiteSetting(
const ContentSettingsPattern& primary_pattern,
const ContentSettingsPattern& secondary_pattern,
ContentSettingsType content_type,
@@ -59,10 +60,10 @@ bool PlatformAppProvider::SetWebsiteSetting(
return false;
}
-void PlatformAppProvider::ClearAllContentSettingsRules(
+void InternalExtensionProvider::ClearAllContentSettingsRules(
ContentSettingsType content_type) {}
-void PlatformAppProvider::Observe(int type,
+void InternalExtensionProvider::Observe(int type,
const content::NotificationSource& source,
const content::NotificationDetails& details) {
switch (type) {
@@ -92,13 +93,13 @@ void PlatformAppProvider::Observe(int type,
}
}
-void PlatformAppProvider::ShutdownOnUIThread() {
+void InternalExtensionProvider::ShutdownOnUIThread() {
DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
RemoveAllObservers();
registrar_.reset();
}
-void PlatformAppProvider::SetContentSettingForExtension(
+void InternalExtensionProvider::SetContentSettingForExtension(
const extensions::Extension* extension,
ContentSetting setting) {
scoped_ptr<ContentSettingsPattern::BuilderInterface> pattern_builder(

Powered by Google App Engine
This is Rietveld 408576698