diff options
| author | jianli@chromium.org <jianli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-22 22:59:12 +0000 |
|---|---|---|
| committer | jianli@chromium.org <jianli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-22 22:59:12 +0000 |
| commit | d746bbe69a40934d99c9ea5822c1203cd4a56150 (patch) | |
| tree | 990ceedd1609c99ae949261d32f086ec1335086e /chrome/browser/notifications | |
| parent | 35a70a431982741d1451b9ac612cd3bc716df232 (diff) | |
| download | chromium_src-d746bbe69a40934d99c9ea5822c1203cd4a56150.zip chromium_src-d746bbe69a40934d99c9ea5822c1203cd4a56150.tar.gz chromium_src-d746bbe69a40934d99c9ea5822c1203cd4a56150.tar.bz2 | |
Add getPermissionLevel and onPermissionLevelChanged to notifications API
BUG=305911
TEST=new tests
Review URL: https://codereview.chromium.org/33173002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@230241 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/notifications')
| -rw-r--r-- | chrome/browser/notifications/desktop_notification_service.cc | 19 | ||||
| -rw-r--r-- | chrome/browser/notifications/desktop_notification_service.h | 4 |
2 files changed, 23 insertions, 0 deletions
diff --git a/chrome/browser/notifications/desktop_notification_service.cc b/chrome/browser/notifications/desktop_notification_service.cc index a910ecb..5b2e7f3 100644 --- a/chrome/browser/notifications/desktop_notification_service.cc +++ b/chrome/browser/notifications/desktop_notification_service.cc @@ -12,6 +12,8 @@ #include "chrome/browser/content_settings/content_settings_details.h" #include "chrome/browser/content_settings/content_settings_provider.h" #include "chrome/browser/content_settings/host_content_settings_map.h" +#include "chrome/browser/extensions/api/notifications/notifications_api.h" +#include "chrome/browser/extensions/event_router.h" #include "chrome/browser/extensions/extension_info_map.h" #include "chrome/browser/extensions/extension_service.h" #include "chrome/browser/extensions/extension_system.h" @@ -587,6 +589,7 @@ void DesktopNotificationService::SetNotifierEnabled( pref_name = prefs::kMessageCenterDisabledExtensionIds; add_new_item = !enabled; id.reset(new base::StringValue(notifier_id.id)); + FirePermissionLevelChangedEvent(notifier_id, enabled); break; case NotifierId::SYSTEM_COMPONENT: #if defined(OS_CHROMEOS) @@ -682,3 +685,19 @@ void DesktopNotificationService::Observe( SetNotifierEnabled(notifier_id, true); } + +void DesktopNotificationService::FirePermissionLevelChangedEvent( + const NotifierId& notifier_id, bool enabled) { + DCHECK_EQ(NotifierId::APPLICATION, notifier_id.type); + extensions::api::notifications::PermissionLevel permission = + enabled ? extensions::api::notifications::PERMISSION_LEVEL_GRANTED + : extensions::api::notifications::PERMISSION_LEVEL_DENIED; + scoped_ptr<base::ListValue> args(new base::ListValue()); + args->Append(new base::StringValue( + extensions::api::notifications::ToString(permission))); + scoped_ptr<extensions::Event> event(new extensions::Event( + extensions::api::notifications::OnPermissionLevelChanged::kEventName, + args.Pass())); + extensions::ExtensionSystem::Get(profile_)->event_router()-> + DispatchEventToExtension(notifier_id.id, event.Pass()); +} diff --git a/chrome/browser/notifications/desktop_notification_service.h b/chrome/browser/notifications/desktop_notification_service.h index 46defd4..cd9dbd0 100644 --- a/chrome/browser/notifications/desktop_notification_service.h +++ b/chrome/browser/notifications/desktop_notification_service.h @@ -195,6 +195,10 @@ class DesktopNotificationService : public BrowserContextKeyedService, // Called when the enabled_sync_notifier_id pref has been changed. void OnEnabledSyncNotifierIdsChanged(); + void FirePermissionLevelChangedEvent( + const message_center::NotifierId& notifier_id, + bool enabled); + // content::NotificationObserver: virtual void Observe(int type, const content::NotificationSource& source, |