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

Unified Diff: chrome/browser/content_settings/content_settings_pref_provider.h

Issue 9826020: Remove code to keep obsolete content settings patterns pref in sync. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix bad file header. Created 8 years, 9 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/content_settings/content_settings_pref_provider.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/content_settings/content_settings_pref_provider.h
diff --git a/chrome/browser/content_settings/content_settings_pref_provider.h b/chrome/browser/content_settings/content_settings_pref_provider.h
index 2b8b2c25418ab85e73b3b21f3e087bb1a77e543a..07220947cce372c3571f1bbcf97b9a2f3ed4e037 100644
--- a/chrome/browser/content_settings/content_settings_pref_provider.h
+++ b/chrome/browser/content_settings/content_settings_pref_provider.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
@@ -88,32 +88,6 @@ class PrefProvider : public ObservableProvider,
const base::Value* value,
DictionaryValue* pattern_pairs_settings);
- // Updates the preferences prefs::kContentSettingsPatterns. This preferences
- // is obsolete and only used for compatibility reasons.
- void UpdateObsoletePatternsPref(
- const ContentSettingsPattern& primary_pattern,
- const ContentSettingsPattern& secondary_pattern,
- ContentSettingsType content_type,
- const ResourceIdentifier& resource_identifier,
- ContentSetting setting);
-
- // Updates the preference prefs::kGeolocationContentSettings. This preference
- // is obsolete and only used to keep sync working with older chrome versions
- // that do not know about the new preference.
- void UpdateObsoleteGeolocationPref(
- const ContentSettingsPattern& primary_pattern,
- const ContentSettingsPattern& secondary_pattern,
- ContentSetting setting);
-
- // Updates the obsolete notifications settings in the passed list values
- // |allowed_sites| and |denied_sites|.
- void UpdateObsoleteNotificationsSettings(
- const ContentSettingsPattern& primary_pattern,
- const ContentSettingsPattern& secondary_pattern,
- ContentSetting setting,
- ListValue* allowed_sites,
- ListValue* denied_sites);
-
// Various migration methods (old cookie, popup and per-host data gets
// migrated to the new format). When calling these functions, |lock_|
// should not be held, since these functions will send out notifications of
@@ -124,20 +98,6 @@ class PrefProvider : public ObservableProvider,
void MigrateObsoleteGeolocationPref();
void MigrateObsoleteNotificationsPrefs();
- // Copies the value of the preference that stores the content settings
- // exceptions to the obsolete preference for content settings exceptions. This
- // is necessary to allow content settings exceptions beeing synced to older
- // versions of chrome that only use the obsolete preference.
- void SyncObsoletePatternPref();
-
- // Copies the notifications and geolocation content settings exceptions from
- // the preference that stores the content settings exceptions to the obsolete
- // preference for notification and geolocation content settings exceptions.
- // This is necessary to allow notifications and geolocation content settings
- // exceptions being synced to older versions of chrome that only use the
- // obsolete preference.
- void SyncObsoletePrefs();
-
static void CanonicalizeContentSettingsExceptions(
base::DictionaryValue* all_settings_dictionary);
« no previous file with comments | « no previous file | chrome/browser/content_settings/content_settings_pref_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698