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

Unified Diff: chrome/browser/ui/content_settings/content_setting_bubble_model.cc

Issue 2502743003: Allow getting both reading/setting cookie settings at one time (Closed)
Patch Set: rdevlin review Created 4 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: chrome/browser/ui/content_settings/content_setting_bubble_model.cc
diff --git a/chrome/browser/ui/content_settings/content_setting_bubble_model.cc b/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
index 077761946b652adccdd5e8a1af6fa2be028068df..5beec856262d3b138ca62d5c706c5a174a1a2933 100644
--- a/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
+++ b/chrome/browser/ui/content_settings/content_setting_bubble_model.cc
@@ -359,8 +359,8 @@ void ContentSettingSingleRadioGroup::SetRadioGroup() {
if (content_type() == CONTENT_SETTINGS_TYPE_COOKIES) {
content_settings::CookieSettings* cookie_settings =
CookieSettingsFactory::GetForProfile(profile()).get();
- setting = cookie_settings->GetCookieSetting(
- url, url, true, &setting_source);
+ cookie_settings->GetCookieSetting(url, url, &setting_source, nullptr,
+ &setting);
} else {
SettingInfo info;
HostContentSettingsMap* map =
« no previous file with comments | « chrome/browser/net/chrome_network_delegate.cc ('k') | components/content_settings/core/browser/cookie_settings.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698