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

Unified Diff: chrome/browser/chrome_content_browser_client.cc

Issue 10447117: Unwire the clear on exit preference from the storage systems. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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
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 eee8d39144f1a0cf67bf8a9dd4d37e4666336617..8a3484b32fb726fb1a1ada241b4ac2e602d08edd 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -914,8 +914,10 @@ bool ChromeContentBrowserClient::AllowSaveLocalState(
content::ResourceContext* context) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
ProfileIOData* io_data = ProfileIOData::FromResourceContext(context);
+ CookieSettings* cookie_settings = io_data->GetCookieSettings();
+ ContentSetting setting = cookie_settings->GetDefaultCookieSetting(NULL);
Bernhard Bauer 2012/05/31 16:08:17 Ooh, now that we're hooking this up to content set
jochen (gone - plz use gerrit) 2012/06/01 12:15:19 Done.
- return !io_data->clear_local_state_on_exit()->GetValue();
+ return setting != CONTENT_SETTING_SESSION_ONLY;
marja 2012/05/31 15:30:32 What if the default setting is .._BLOCK?
jochen (gone - plz use gerrit) 2012/05/31 15:54:41 It's a bug that this doesn't keep flash from savin
}
bool ChromeContentBrowserClient::AllowWorkerDatabase(
« no previous file with comments | « no previous file | chrome/browser/net/chrome_url_request_context.h » ('j') | chrome/browser/net/sqlite_persistent_cookie_store.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698