Introduce the Blink NotificationService, move permission checks there

Introduce the Blink NotificationService, move permission checks there

This introduces the Mojo NotificationService in Blink and moves
permission checks over from the IPC code-path to the Mojo code-path.

This CL is a re-land of the original CL that was reverted because of
crash reports: https://codereview.chromium.org/1948223004/. In
addition, it removes usage of WeakPtr as pointed out by yszhen@.

TBR=esprehn, jam, dcheng, mvanouwerkerk, haraken
BUG=595685
Committed: https://crrev.com/f28cb7f78b2f5df0d02ccdfa430edf5e304dfcdd
Cr-Commit-Position: refs/heads/master@{#397701}

[Peter Beverloo, 2016-06-03 13:22:15]

Description was changed from

==========
Introduce the Blink NotificationService, move permission checks there

This introduces the Mojo NotificationService in Blink and moves
permission checks over from the IPC code-path to the Mojo code-path.

This CL is a re-land of the original CL that was reverted because of
crash reports: https://codereview.chromium.org/1948223004/

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

to

==========
Introduce the Blink NotificationService, move permission checks there

This introduces the Mojo NotificationService in Blink and moves
permission checks over from the IPC code-path to the Mojo code-path.

This CL is a re-land of the original CL that was reverted because of
crash reports: https://codereview.chromium.org/1948223004/. In
addition, it removes usage of WeakPtr as pointed out by yszhen@.

TBR=esprehn, jam, dcheng, mvanouwerkerk, haraken
BUG=595685
==========

[Peter Beverloo, 2016-06-03 13:22:15]

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

[Peter Beverloo, 2016-06-03 13:28:15]

TBR the original reviewers. This CL is identical to the one that landed (albeit
rebased), but with the change suggested by yszhen@ included. Diff visible
between PS1 and 2.

[Peter Beverloo, 2016-06-03 13:28:19]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2016-06-03 13:28:33]

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

[commit-bot: I haz the power, 2016-06-03 14:09:37]

Description was changed from

==========
Introduce the Blink NotificationService, move permission checks there

This introduces the Mojo NotificationService in Blink and moves
permission checks over from the IPC code-path to the Mojo code-path.

This CL is a re-land of the original CL that was reverted because of
crash reports: https://codereview.chromium.org/1948223004/. In
addition, it removes usage of WeakPtr as pointed out by yszhen@.

TBR=esprehn, jam, dcheng, mvanouwerkerk, haraken
BUG=595685
==========

to

==========
Introduce the Blink NotificationService, move permission checks there

This introduces the Mojo NotificationService in Blink and moves
permission checks over from the IPC code-path to the Mojo code-path.

This CL is a re-land of the original CL that was reverted because of
crash reports: https://codereview.chromium.org/1948223004/. In
addition, it removes usage of WeakPtr as pointed out by yszhen@.

TBR=esprehn, jam, dcheng, mvanouwerkerk, haraken
BUG=595685
==========

[commit-bot: I haz the power, 2016-06-03 14:09:39]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-03 14:11:48]

Description was changed from

==========
Introduce the Blink NotificationService, move permission checks there

This introduces the Mojo NotificationService in Blink and moves
permission checks over from the IPC code-path to the Mojo code-path.

This CL is a re-land of the original CL that was reverted because of
crash reports: https://codereview.chromium.org/1948223004/. In
addition, it removes usage of WeakPtr as pointed out by yszhen@.

TBR=esprehn, jam, dcheng, mvanouwerkerk, haraken
BUG=595685
==========

to

==========
Introduce the Blink NotificationService, move permission checks there

This introduces the Mojo NotificationService in Blink and moves
permission checks over from the IPC code-path to the Mojo code-path.

This CL is a re-land of the original CL that was reverted because of
crash reports: https://codereview.chromium.org/1948223004/. In
addition, it removes usage of WeakPtr as pointed out by yszhen@.

TBR=esprehn, jam, dcheng, mvanouwerkerk, haraken
BUG=595685

Committed: https://crrev.com/f28cb7f78b2f5df0d02ccdfa430edf5e304dfcdd
Cr-Commit-Position: refs/heads/master@{#397701}
==========

[commit-bot: I haz the power, 2016-06-03 14:11:50]

Patchset 3 (id:??) landed as
https://crrev.com/f28cb7f78b2f5df0d02ccdfa430edf5e304dfcdd
Cr-Commit-Position: refs/heads/master@{#397701}

[haraken, 2016-06-03 14:15:08]

LGTM