Chromium Code Reviews
DescriptionFix PushManager.subscribe() permission request on Android
https://codereview.chromium.org/2149883002 merged the
PushMessagingPermissionContext into the NotificationPermissionContext.
Unfortunately, this meant that PushManager.subscribe() tried to show
a push messaging permission infobar instead of the notifications
infobar, which fails because there is no push messaging infobar. This
patch switches push messaging back to the notifications infobar.
BUG=644256
Review-Url: https://codereview.chromium.org/2316673002
Cr-Commit-Position: refs/heads/master@{#416762}
(cherry picked from commit e0db126dd99d297686d49eedd745d196708167ee)
Committed: https://chromium.googlesource.com/chromium/src/+/d256ccde6cb75b87e4346d64e6e868028197ab6b
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
||||||||||||||||||||||||||||