Introduce new content APIs for persistent notifications.

Introduce new content APIs for persistent notifications.

This patch introduces two new APIs on the PlatformNotificationService
for displaying and closing persistent notifications. Whilst the message
filter calls them, there are no implementations yet.

BUG=432527
Committed: https://crrev.com/06d77721fe454f99e8e590daea3da0809c56c686
Cr-Commit-Position: refs/heads/master@{#307708}

[Peter Beverloo, 2014-12-08 18:19:10]

peter@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org

[Peter Beverloo, 2014-12-08 18:19:10]

+mvanouwerkerk for initial review.

This CL depends on https://codereview.chromium.org/774573003/.

[Michael van Ouwerkerk, 2014-12-08 19:40:11]

lgtm with nit

https://codereview.chromium.org/776813004/diff/1/content/browser/notification...
File content/browser/notifications/notification_message_filter.cc (right):

https://codereview.chromium.org/776813004/diff/1/content/browser/notification...
content/browser/notifications/notification_message_filter.cc:65:
GetContentClient()->browser()->GetPlatformNotificationService();
I guess we're on the UI thread here? Let's add a check for clarity. Really the
same comment for every method in this class.

[Peter Beverloo, 2014-12-09 18:39:26]

https://codereview.chromium.org/776813004/diff/1/content/browser/notification...
File content/browser/notifications/notification_message_filter.cc (right):

https://codereview.chromium.org/776813004/diff/1/content/browser/notification...
content/browser/notifications/notification_message_filter.cc:65:
GetContentClient()->browser()->GetPlatformNotificationService();
On 2014/12/08 19:40:11, Michael van Ouwerkerk wrote:
> I guess we're on the UI thread here? Let's add a check for clarity. Really the
> same comment for every method in this class.

I reverted this change for now.

[Peter Beverloo, 2014-12-09 19:41:28]

peter@chromium.org changed reviewers:
+ dewittj@chromium.org

[Peter Beverloo, 2014-12-09 19:41:29]

+dewittj

[dewittj, 2014-12-09 20:59:47]

lgtm

[Peter Beverloo, 2014-12-09 21:54:39]

peter@chromium.org changed reviewers:
+ avi@chromium.org

[Peter Beverloo, 2014-12-09 21:54:40]

Thanks! +avi for content/

[Avi (use Gerrit), 2014-12-10 00:19:18]

lgtm

stampity stamp

[Peter Beverloo, 2014-12-10 15:15:30]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2014-12-10 15:16:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/776813004/20001

[commit-bot: I haz the power, 2014-12-10 16:11:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-12-10 16:12:43]

Patchset 2 (id:??) landed as
https://crrev.com/06d77721fe454f99e8e590daea3da0809c56c686
Cr-Commit-Position: refs/heads/master@{#307708}