Some random Web Notification cleanups

Some random Web Notification cleanups

This CL cleans up some discrepancies in the Web Notification module
and removes unused includes. Furthermore, it:

- Merges the ServiceWorkerNotifications and Notifications runtime
  enabled features. The SWN flag is not being toggled anywhere.
- Removes the notificationerror event from the Service Worker
  global scope. We have never supported this event.

BUG=

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

[whywhat, 2015-08-27 19:16:46]

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

[whywhat, 2015-08-27 19:16:47]

lgtm

https://codereview.chromium.org/1321453007/diff/1/LayoutTests/http/tests/noti...
File
LayoutTests/http/tests/notifications/resources/serviceworker-notification-event.js
(right):

https://codereview.chromium.org/1321453007/diff/1/LayoutTests/http/tests/noti...
LayoutTests/http/tests/notifications/resources/serviceworker-notification-event.js:30:
}, 'The notificationclick event exist on the global scope.');
nit: exists

[Peter Beverloo, 2015-08-27 19:19:59]

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

[Peter Beverloo, 2015-08-27 19:20:00]

Thanks Anton!

+adamk from the OWNERS round robin for platform/ and web/ (2 lines) :-).

https://codereview.chromium.org/1321453007/diff/1/LayoutTests/http/tests/noti...
File
LayoutTests/http/tests/notifications/resources/serviceworker-notification-event.js
(right):

https://codereview.chromium.org/1321453007/diff/1/LayoutTests/http/tests/noti...
LayoutTests/http/tests/notifications/resources/serviceworker-notification-event.js:30:
}, 'The notificationclick event exist on the global scope.');
On 2015/08/27 19:16:47, whywhat wrote:
> nit: exists

Done.

[adamk, 2015-08-28 00:20:00]

lgtm for Source/{web,platform} (didn't look at anything else)

[Peter Beverloo, 2015-08-28 11:29:32]

The CQ bit was checked by peter@chromium.org

[Peter Beverloo, 2015-08-28 11:29:32]

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

[commit-bot: I haz the power, 2015-08-28 11:29:42]

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

[commit-bot: I haz the power, 2015-08-28 11:34:32]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201394