Define a Mojo Notification structure in the Blink API

Define a Mojo Notification structure in the Blink API

This will replace the PlatformNotification{Action, Data} structures
that currently live in //content. It's based on a similar CL for Web
Bluetooth: https://codereview.chromium.org/1784613002/

The stack will be switched over in a next CL. Since it's part of a
larger-scale refactoring, I'd like to land this separately to unblock
other people that will depend on this.

BUG=595685
Committed: https://crrev.com/2263dd5457199da3e51086903f0a1caec6bd86d4
Cr-Commit-Position: refs/heads/master@{#382564}

[Peter Beverloo, 2016-03-17 14:03:32]

Description was changed from

==========
Define a Mojo Notification structure in the Blink API

This will replace the PlatformNotification{Action, Data} structures
that currently live in //content. It's based on a similar CL for Web
Bluetooth: https://codereview.chromium.org/1784613002/

BUG=595685
==========

to

==========
Define a Mojo Notification structure in the Blink API

This will replace the PlatformNotification{Action, Data} structures
that currently live in //content. It's based on a similar CL for Web
Bluetooth: https://codereview.chromium.org/1784613002/

The stack will be switched over in a next CL. Since it's part of a
larger-scale refactoring, I'd like to land this separately to unblock
other people that will depend on this.

BUG=595685
==========

[Peter Beverloo, 2016-03-17 14:24:54]

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

[Peter Beverloo, 2016-03-17 14:24:55]

+mvanouwerkerk for notifications
+esprehn for the API

Elliott, until modules can use use Mojo and //base(/time) directly, the
Notification module will work with this class through a PlatformNotification
proxy class in Source/platform/. Chromium's usage usage will be straightforward.

A Service will be defined in the API as a second step - we have to move quite
some other stuff down to Blink before that can happen. That doesn't have to stop
the data :-).

[Michael van Ouwerkerk, 2016-03-17 14:38:30]

lgtm ship it

[esprehn, 2016-03-17 20:19:07]

You can depend on mojo today, see the blink-dev email for how to generate WTF
friendly bindings. I don't think base::Time is coming to modules, the web
represents times as doubles. Why do you need it? I think you want to put
whatever you're doing with time in WTF or Platform.

[Peter Beverloo, 2016-03-18 17:55:34]

Because Chrome doesn't use time as a double. Nonetheless, it doesn't matter
whether I do the conversion within Blink or outside of it.

PTAL

[Peter Beverloo, 2016-03-21 15:12:52]

Elliott: Friendly ping.

[esprehn, 2016-03-21 20:32:44]

lgtm

[Peter Beverloo, 2016-03-22 10:41:47]

The CQ bit was checked by peter@chromium.org

[Peter Beverloo, 2016-03-22 10:41:47]

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

[commit-bot: I haz the power, 2016-03-22 10:41:52]

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

[commit-bot: I haz the power, 2016-03-22 13:35:29]

Description was changed from

==========
Define a Mojo Notification structure in the Blink API

This will replace the PlatformNotification{Action, Data} structures
that currently live in //content. It's based on a similar CL for Web
Bluetooth: https://codereview.chromium.org/1784613002/

The stack will be switched over in a next CL. Since it's part of a
larger-scale refactoring, I'd like to land this separately to unblock
other people that will depend on this.

BUG=595685
==========

to

==========
Define a Mojo Notification structure in the Blink API

This will replace the PlatformNotification{Action, Data} structures
that currently live in //content. It's based on a similar CL for Web
Bluetooth: https://codereview.chromium.org/1784613002/

The stack will be switched over in a next CL. Since it's part of a
larger-scale refactoring, I'd like to land this separately to unblock
other people that will depend on this.

BUG=595685
==========

[commit-bot: I haz the power, 2016-03-22 13:35:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-22 13:36:34]

Description was changed from

==========
Define a Mojo Notification structure in the Blink API

This will replace the PlatformNotification{Action, Data} structures
that currently live in //content. It's based on a similar CL for Web
Bluetooth: https://codereview.chromium.org/1784613002/

The stack will be switched over in a next CL. Since it's part of a
larger-scale refactoring, I'd like to land this separately to unblock
other people that will depend on this.

BUG=595685
==========

to

==========
Define a Mojo Notification structure in the Blink API

This will replace the PlatformNotification{Action, Data} structures
that currently live in //content. It's based on a similar CL for Web
Bluetooth: https://codereview.chromium.org/1784613002/

The stack will be switched over in a next CL. Since it's part of a
larger-scale refactoring, I'd like to land this separately to unblock
other people that will depend on this.

BUG=595685

Committed: https://crrev.com/2263dd5457199da3e51086903f0a1caec6bd86d4
Cr-Commit-Position: refs/heads/master@{#382564}
==========

[commit-bot: I haz the power, 2016-03-22 13:36:35]

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