The "push" event should inherit from ExtendableEvent.

The "push" event should inherit from ExtendableEvent.

This makes it possible for developers to call waitUntil() on the
event, in case its lifetime should be extended.

Specification link (Push API):
  https://w3c.github.io/push-api/#the-push-event


BUG=430888
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185374

[Peter Beverloo, 2014-11-12 17:34:44]

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

[Peter Beverloo, 2014-11-12 17:34:44]

+mvanouwerkerk for Push

This depends on https://codereview.chromium.org/715013003/.

[Michael van Ouwerkerk, 2014-11-12 17:49:33]

lgtm

My goodness.

[Peter Beverloo, 2014-11-13 14:56:24]

peter@chromium.org changed reviewers:
+ michaeln@chromium.org, mkwst@chromium.org

[Peter Beverloo, 2014-11-13 14:56:25]

+mvanouwerkerk - Please re-check the test and Exposed= change.
+michaeln - Service Workers changes.
+mkwst - Source/web/ RS.

The three Mikes :-).

[Michael van Ouwerkerk, 2014-11-13 15:03:08]

still lgtm

[Mike West, 2014-11-13 15:16:44]

Source/web LGTM.

[falken, 2014-11-14 02:30:23]

falken@chromium.org changed reviewers:
+ falken@chromium.org

[falken, 2014-11-14 02:30:24]

lgtm for service worker

Service Worker patches link to the spec or spec discussion for IDL changes like
this in the description, would it make sense to do so here as well?

[Peter Beverloo, 2014-11-14 09:26:10]

On 2014/11/14 02:30:24, falken wrote:
> lgtm for service worker
> 
> Service Worker patches link to the spec or spec discussion for IDL changes
like
> this in the description, would it make sense to do so here as well?

Yeah, I like that. Thanks for the suggestion, done.

I'll submit this after the other CL has landed.

[Peter Beverloo, 2014-11-14 11:57:24]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2014-11-14 11:57:54]

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

[commit-bot: I haz the power, 2014-11-14 12:55:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-14 12:55:17]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/3...)

[Peter Beverloo, 2014-11-14 13:08:28]

The CQ bit was checked by peter@chromium.org

[commit-bot: I haz the power, 2014-11-14 13:10:50]

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

[commit-bot: I haz the power, 2014-11-14 14:35:18]

Committed patchset #6 (id:100001) as 185374