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

Unified Diff: ios/chrome/browser/content_settings/host_content_settings_map_factory.cc

Issue 2078893002: Add callback list to PrefModelAssociator after sync data is loaded (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@migrate_domain_scoped_settings
Patch Set: Created 4 years, 6 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: ios/chrome/browser/content_settings/host_content_settings_map_factory.cc
diff --git a/ios/chrome/browser/content_settings/host_content_settings_map_factory.cc b/ios/chrome/browser/content_settings/host_content_settings_map_factory.cc
index e6faf69a0000dce8d0efafdbfb15672ff67c34cd..369669ace4070aefe9168cfbb86a90996363a2b8 100644
--- a/ios/chrome/browser/content_settings/host_content_settings_map_factory.cc
+++ b/ios/chrome/browser/content_settings/host_content_settings_map_factory.cc
@@ -44,7 +44,7 @@ HostContentSettingsMapFactory::BuildServiceInstanceFor(
}
return make_scoped_refptr(new HostContentSettingsMap(
browser_state->GetPrefs(), browser_state->IsOffTheRecord(),
- false /* guest_profile */));
+ false /* guest_profile */, nullptr /* PrefServiceSyncable* */));
raymes 2016/06/29 03:50:01 nit: same here
lshang 2016/06/30 05:03:36 Same as above, don't need to address this after we
}
web::BrowserState* HostContentSettingsMapFactory::GetBrowserStateToUse(

Powered by Google App Engine
This is Rietveld 408576698