| commit | b514ab0de3c448941a4b3d16d5984b042e3d51c0 | |
| author | peter <peter@chromium.org> | |
| Thu, 2 Jul 2015 12:10:51 +0000 (2 05:10 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Thu, 2 Jul 2015 12:11:29 +0000 (2 12:11 +0000) | ||
| tree | 14f118b1b98f21a66be5109e818b85e59fd0b5da | treesnapshot (tar.gz zip) |
| parent | 56116f4d5ff944b3f942fbb451911f78d8f8b0af | commitdiff |
Simplify permission-related code for Web Notifications.
(1) Remove support for inheriting the "notification" API permission
for packaged and hosted apps using the Web Notification API. This
has been inadvertently broken since Chrome 42.
(2) Give notifications a dedicated NotificationPermissionContext,
similar to those other features have. (+ unit tests)
(3) Remove and clean up all unused code.
This further emphasizes the DesktopNotificationService as a deprecated
concept. I'll remove it entirely in a follow-up patch.
BUG=504361
Review URL: https://codereview.chromium.org/1207363002
Cr-Commit-Position: refs/heads/master@{#337206}
(1) Remove support for inheriting the "notification" API permission
for packaged and hosted apps using the Web Notification API. This
has been inadvertently broken since Chrome 42.
(2) Give notifications a dedicated NotificationPermissionContext,
similar to those other features have. (+ unit tests)
(3) Remove and clean up all unused code.
This further emphasizes the DesktopNotificationService as a deprecated
concept. I'll remove it entirely in a follow-up patch.
BUG=504361
Review URL: https://codereview.chromium.org/1207363002
Cr-Commit-Position: refs/heads/master@{#337206}
15 files changed: