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

Unified Diff: components/content_settings/core/browser/host_content_settings_map.h

Issue 1411653004: Remove last usage functions from HostContentSettingsMap Base URL: https://chromium.googlesource.com/chromium/src.git@simplify-host-content-settings
Patch Set: Created 5 years, 1 month 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: components/content_settings/core/browser/host_content_settings_map.h
diff --git a/components/content_settings/core/browser/host_content_settings_map.h b/components/content_settings/core/browser/host_content_settings_map.h
index 3eeafe48ca1024920d6b2f15445095e4192246bc..29d568224f841384a578461160481a4370916208 100644
--- a/components/content_settings/core/browser/host_content_settings_map.h
+++ b/components/content_settings/core/browser/host_content_settings_map.h
@@ -189,42 +189,6 @@ class HostContentSettingsMap : public content_settings::Observer,
return is_off_the_record_;
}
- // Returns a single |ContentSetting| which applies to the given URLs, just as
- // |GetContentSetting| does. If the setting is allowed, it also records the
- // last usage to preferences.
- //
- // This should only be called on the UI thread, unlike |GetContentSetting|.
- ContentSetting GetContentSettingAndMaybeUpdateLastUsage(
- const GURL& primary_url,
- const GURL& secondary_url,
- ContentSettingsType content_type,
- const std::string& resource_identifier);
-
- // Sets the last time that a given content type has been used for the pattern
- // which matches the URLs to the current time.
- void UpdateLastUsage(const GURL& primary_url,
- const GURL& secondary_url,
- ContentSettingsType content_type);
-
- // Sets the last time that a given content type has been used for a pattern
- // pair to the current time.
- void UpdateLastUsageByPattern(const ContentSettingsPattern& primary_pattern,
- const ContentSettingsPattern& secondary_pattern,
- ContentSettingsType content_type);
-
- // Returns the last time the pattern that matches the URL has requested
- // permission for the |content_type| setting.
- base::Time GetLastUsage(const GURL& primary_url,
- const GURL& secondary_url,
- ContentSettingsType content_type);
-
- // Returns the last time the pattern has requested permission for the
- // |content_type| setting.
- base::Time GetLastUsageByPattern(
- const ContentSettingsPattern& primary_pattern,
- const ContentSettingsPattern& secondary_pattern,
- ContentSettingsType content_type);
-
// Adds/removes an observer for content settings changes.
void AddObserver(content_settings::Observer* observer);
void RemoveObserver(content_settings::Observer* observer);

Powered by Google App Engine
This is Rietveld 408576698