Move ContentSettingsDetails and Pattern out of HostContentSettingsMap as separate classes.

chrome/browser/ui/views/options/content_filter_page_view.cc

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..e272cd876c4c2071c17cfc3aeccd2be739a565ce 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) {

[markusheintz_, 2010/12/02 13:28:18]

s/ContentSettingsDetails *details/ContentSettingsDetails* details/g

   if (details->type() == CONTENT_SETTINGS_TYPE_DEFAULT ||
       details->type() == content_type_) {
     UpdateView();
@@ -265,7 +266,7 @@ void ContentFilterPageView::Observe(NotificationType type,
                        const NotificationDetails& details) {
   if (type == NotificationType::CONTENT_SETTINGS_CHANGED) {
     NotifyContentSettingsChanged(

[gfeher, 2010/12/02 13:07:50]

This actually fits into 80 chars.

[jochen (gone - plz use gerrit), 2010/12/02 13:18:43]

almost :)

-        Details<HostContentSettingsMap::ContentSettingsDetails>
+        Details<ContentSettingsDetails>
             (details).ptr());
   } else {
     OptionsPageBase::Observe(type, source, details);