diff options
| author | peter <peter@chromium.org> | 2015-07-02 05:10:51 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-07-02 12:11:29 +0000 |
| commit | b514ab0de3c448941a4b3d16d5984b042e3d51c0 (patch) | |
| tree | 14f118b1b98f21a66be5109e818b85e59fd0b5da /chrome/chrome_browser.gypi | |
| parent | 56116f4d5ff944b3f942fbb451911f78d8f8b0af (diff) | |
| download | chromium_src-b514ab0de3c448941a4b3d16d5984b042e3d51c0.zip chromium_src-b514ab0de3c448941a4b3d16d5984b042e3d51c0.tar.gz chromium_src-b514ab0de3c448941a4b3d16d5984b042e3d51c0.tar.bz2 | |
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}
Diffstat (limited to 'chrome/chrome_browser.gypi')
| -rw-r--r-- | chrome/chrome_browser.gypi | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/chrome/chrome_browser.gypi b/chrome/chrome_browser.gypi index 43001e3..278e0bd 100644 --- a/chrome/chrome_browser.gypi +++ b/chrome/chrome_browser.gypi @@ -1980,6 +1980,10 @@ 'browser/notifications/notification.cc', 'browser/notifications/notification.h', 'browser/notifications/notification_delegate.h', + 'browser/notifications/notification_permission_context.cc', + 'browser/notifications/notification_permission_context.h', + 'browser/notifications/notification_permission_context_factory.cc', + 'browser/notifications/notification_permission_context_factory.h', 'browser/notifications/notification_object_proxy.cc', 'browser/notifications/notification_object_proxy.h', 'browser/notifications/notification_permission_infobar_delegate.cc', |