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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 2888373003: Clean up unused ContentBrowserClient methods that access cookie settings. (Closed)
Patch Set: Created 3 years, 7 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 | « chrome/browser/chrome_content_browser_client.h ('k') | content/public/browser/content_browser_client.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 0738496bfc2a5ce97414bf885c594f7055105be4..a89cd9904b2f9942d23335eaf4e0dc39c901914a 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -2120,19 +2120,6 @@ bool ChromeContentBrowserClient::AllowSetCookie(
return allow;
}
-bool ChromeContentBrowserClient::AllowSaveLocalState(
- content::ResourceContext* context) {
- DCHECK_CURRENTLY_ON(BrowserThread::IO);
- ProfileIOData* io_data = ProfileIOData::FromResourceContext(context);
- content_settings::CookieSettings* cookie_settings =
- io_data->GetCookieSettings();
- ContentSetting setting = cookie_settings->GetDefaultCookieSetting(NULL);
-
- // TODO(bauerb): Should we also disallow local state if the default is BLOCK?
- // Could we even support per-origin settings?
- return setting != CONTENT_SETTING_SESSION_ONLY;
-}
-
void ChromeContentBrowserClient::AllowWorkerFileSystem(
const GURL& url,
content::ResourceContext* context,
@@ -2243,19 +2230,6 @@ bool ChromeContentBrowserClient::AllowWorkerIndexedDB(
return allow;
}
-#if BUILDFLAG(ENABLE_WEBRTC)
-bool ChromeContentBrowserClient::AllowWebRTCIdentityCache(
- const GURL& url,
- const GURL& first_party_url,
- content::ResourceContext* context) {
- DCHECK_CURRENTLY_ON(BrowserThread::IO);
- ProfileIOData* io_data = ProfileIOData::FromResourceContext(context);
- content_settings::CookieSettings* cookie_settings =
- io_data->GetCookieSettings();
- return cookie_settings->IsCookieAccessAllowed(url, first_party_url);
-}
-#endif // BUILDFLAG(ENABLE_WEBRTC)
-
ChromeContentBrowserClient::AllowWebBluetoothResult
ChromeContentBrowserClient::AllowWebBluetooth(
content::BrowserContext* browser_context,
« no previous file with comments | « chrome/browser/chrome_content_browser_client.h ('k') | content/public/browser/content_browser_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698