Implement the ability to get all notifications in Chromium.

Implement the ability to get all notifications in Chromium.

This patch implements the SWR.getNotifications() API in Chromium,
by hooking it up to the Notification database. All functionality,
including the filter-by-tag option, are supported.

This CL is part of a two-sided patch:
  [1] This CL.
  [2] https://codereview.chromium.org/1056573002/

BUG=442143
Committed: https://crrev.com/08f04f149112e624606dda88d63951942348c5a4
Cr-Commit-Position: refs/heads/master@{#325626}

[Peter Beverloo, 2015-04-01 16:32:18]

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

[Peter Beverloo, 2015-04-01 16:32:19]

+johnme for notifications

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

[johnme, 2015-04-08 18:12:41]

lgtm with nit, thanks :)

https://codereview.chromium.org/1057573002/diff/20001/content/browser/notific...
File content/browser/notifications/platform_notification_context_impl.cc
(right):

https://codereview.chromium.org/1057573002/diff/20001/content/browser/notific...
content/browser/notifications/platform_notification_context_impl.cc:147: //
TODO(peter): Record UMA on |status| for reading from the database.
Since you wrote this patch, the other methods now log UMA. When you rebase this,
consider logging to Notifications.Database.ReadResult ?

[Peter Beverloo, 2015-04-14 18:01:27]

peter@chromium.org changed reviewers:
+ askvitkine@chromium.org, rkaplow@chromium.org

[Peter Beverloo, 2015-04-14 18:01:28]

Done!

+askvitkine and rkaplow for UMA additions

https://codereview.chromium.org/1057573002/diff/20001/content/browser/notific...
File content/browser/notifications/platform_notification_context_impl.cc
(right):

https://codereview.chromium.org/1057573002/diff/20001/content/browser/notific...
content/browser/notifications/platform_notification_context_impl.cc:147: //
TODO(peter): Record UMA on |status| for reading from the database.
On 2015/04/08 18:12:41, johnme wrote:
> Since you wrote this patch, the other methods now log UMA. When you rebase
this,
> consider logging to Notifications.Database.ReadResult ?

Done.

[Peter Beverloo, 2015-04-16 17:03:02]

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

[Peter Beverloo, 2015-04-16 17:03:02]

+avi for //content/public/ (API will be used by PushMessagingServiceImpl.)

+askvitkine, rkaplow for histogram addition.

[Peter Beverloo, 2015-04-16 17:04:14]

peter@chromium.org changed reviewers:
+ asvitkine@google.com
- askvitkine@chromium.org

[Peter Beverloo, 2015-04-16 17:04:14]

-askvitkine, +asvitkine

The askvitkine account is a bit confusing..

[Peter Beverloo, 2015-04-16 17:05:06]

peter@chromium.org changed reviewers:
+ asvitkine@chromium.org
- asvitkine@google.com

[Alexei Svitkine (slow), 2015-04-16 17:06:52]

lgtm

[Avi (use Gerrit), 2015-04-16 18:53:27]

content api lgtm

[Peter Beverloo, 2015-04-17 11:01:03]

The CQ bit was checked by peter@chromium.org

[Peter Beverloo, 2015-04-17 11:01:03]

The patchset sent to the CQ was uploaded after l-g-t-m from johnme@chromium.org
Link to the patchset: https://codereview.chromium.org/1057573002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2015-04-17 11:01:28]

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

[Peter Beverloo, 2015-04-17 11:02:13]

The CQ bit was unchecked by peter@chromium.org

[Peter Beverloo, 2015-04-17 12:58:19]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2015-04-17 12:58:28]

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

[commit-bot: I haz the power, 2015-04-17 13:52:40]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-04-17 13:53:26]

Patchset 4 (id:??) landed as
https://crrev.com/08f04f149112e624606dda88d63951942348c5a4
Cr-Commit-Position: refs/heads/master@{#325626}