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

Unified Diff: chrome/browser/ui/views/options/content_filter_page_view.cc

Issue 5574001: Move ContentSettingsDetails and Pattern out of HostContentSettingsMap as separate classes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/chrome/browser/content_settings
Patch Set: updates Created 10 years 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/ui/views/options/content_filter_page_view.cc
diff --git a/chrome/browser/ui/views/options/content_filter_page_view.cc b/chrome/browser/ui/views/options/content_filter_page_view.cc
index 320611d68e3269d4ca8ae6071bad2ff1370b884f..ebceafa2aeed2b19224973f567d1c6abb639a4c8 100644
--- a/chrome/browser/ui/views/options/content_filter_page_view.cc
+++ b/chrome/browser/ui/views/options/content_filter_page_view.cc
@@ -6,6 +6,7 @@
#include "app/l10n_util.h"
#include "base/command_line.h"
+#include "chrome/browser/content_settings/content_settings_details.h"
#include "chrome/browser/geolocation/geolocation_content_settings_map.h"
#include "chrome/browser/geolocation/geolocation_exceptions_table_model.h"
#include "chrome/browser/notifications/desktop_notification_service.h"
@@ -253,7 +254,7 @@ void ContentFilterPageView::ButtonPressed(views::Button* sender,
}
void ContentFilterPageView::NotifyContentSettingsChanged(
- const HostContentSettingsMap::ContentSettingsDetails *details) {
+ const ContentSettingsDetails* details) {
if (details->type() == CONTENT_SETTINGS_TYPE_DEFAULT ||
details->type() == content_type_) {
UpdateView();
@@ -265,8 +266,7 @@ void ContentFilterPageView::Observe(NotificationType type,
const NotificationDetails& details) {
if (type == NotificationType::CONTENT_SETTINGS_CHANGED) {
NotifyContentSettingsChanged(
- Details<HostContentSettingsMap::ContentSettingsDetails>
- (details).ptr());
+ Details<ContentSettingsDetails>(details).ptr());
} else {
OptionsPageBase::Observe(type, source, details);
}

Powered by Google App Engine
This is Rietveld 408576698