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

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

Issue 2886433002: [Android] Adding content settings provider for notification channels (Closed)
Patch Set: Responding to comments 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
Index: chrome/browser/content_settings/host_content_settings_map_factory.cc
diff --git a/chrome/browser/content_settings/host_content_settings_map_factory.cc b/chrome/browser/content_settings/host_content_settings_map_factory.cc
index a3465561e0b82d26889314daa44e73579075289e..9a424c732cdc0487b66d4e5ebff0b7cb38986cb1 100644
--- a/chrome/browser/content_settings/host_content_settings_map_factory.cc
+++ b/chrome/browser/content_settings/host_content_settings_map_factory.cc
@@ -7,6 +7,7 @@
#include <utility>
#include "base/feature_list.h"
+#include "chrome/browser/notifications/notification_content_settings_provider_android.h"
#include "chrome/browser/prefs/pref_service_syncable_util.h"
#include "chrome/browser/profiles/off_the_record_profile_impl.h"
#include "chrome/browser/profiles/profile.h"
@@ -109,6 +110,13 @@ scoped_refptr<RefcountedKeyedService>
}
#endif // BUILDFLAG(ENABLE_SUPERVISED_USERS)
+#if defined(OS_ANDROID)
Peter Beverloo 2017/05/16 16:15:34 Should we also guard the implementation behind a b
raymes 2017/05/17 03:13:38 +1 good idea :)
awdf 2017/05/17 17:14:40 Done.
+ auto android_system_provider =
Peter Beverloo 2017/05/16 16:15:34 micro nit: rename
awdf 2017/05/17 17:14:40 Done.
+ base::MakeUnique<NotificationContentSettingsProviderAndroid>();
+ settings_map->RegisterProvider(
+ HostContentSettingsMap::NOTIFICATION_CONTENT_SETTINGS_ANDROID_PROVIDER,
+ std::move(android_system_provider));
+#endif
return settings_map;
}

Powered by Google App Engine
This is Rietveld 408576698