DescriptionPush API: Forced notifications should use Notifications database
1. Writes to the Notifications database when creating forced notifications.
2. Uses Notifications database when counting notifications, instead of the unreliable NotificationUIManager APIs.
This gives more reliable values after browser restarts.
BUG=437277
TBR=avi@chromium.org
Committed: https://crrev.com/830ecef7ea5e4fb162fb7cfdc6be2cbe4fd1d52b
Cr-Commit-Position: refs/heads/master@{#327716}
Patch Set 1 #
Total comments: 2
Patch Set 2 : Fix threading #Patch Set 3 : Also write forced notifications to database #Patch Set 4 : Don't use BindToCurrentLoop yet #
Total comments: 10
Patch Set 5 : Address Peter's review nits #
Messages
Total messages: 18 (6 generated)
|