Stop requiring NotificationAction action and title to be non-empty

Stop requiring NotificationAction action and title to be non-empty

Improve spec compliance. Note that
http/tests/notifications/serviceworkerregistration-document-actions-throw.html
still tests that they are set (but it's no longer possible to have an
equivalent check in NotificationDataTest, since required WebIDL
attributes are enforced at the V8 level by [1], and are bypassed when
creating IDL objects from C++).

[1]: https://chromium.googlesource.com/chromium/src/+/dfd8ec1b3edb7f876235bec1138ecec4f2da748d/third_party/WebKit/Source/bindings/templates/dictionary_v8.cpp#55

BUG=557624
Committed: https://crrev.com/56c5368ca167742af3dfdd0bf8f879984b2ad05b
Cr-Commit-Position: refs/heads/master@{#362469}

[johnme, 2015-12-01 17:26:56]

The CQ bit was checked by johnme@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-12-01 17:28:18]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1487063002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1487063002/1

[johnme, 2015-12-01 17:36:46]

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

[Peter Beverloo, 2015-12-01 17:38:52]

lgtm, thank you!

[johnme, 2015-12-01 17:40:32]

The CQ bit was unchecked by johnme@chromium.org

[johnme, 2015-12-01 17:40:38]

The CQ bit was checked by johnme@chromium.org

[commit-bot: I haz the power, 2015-12-01 17:40:50]

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

[commit-bot: I haz the power, 2015-12-01 19:18:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-01 19:19:33]

Description was changed from

==========
Stop requiring NotificationAction action and title to be non-empty

Improve spec compliance. Note that
http/tests/notifications/serviceworkerregistration-document-actions-throw.html
still tests that they are set (but it's no longer possible to have an
equivalent check in NotificationDataTest, since required WebIDL
attributes are enforced at the V8 level by [1], and are bypassed when
creating IDL objects from C++).

[1]:
https://chromium.googlesource.com/chromium/src/+/dfd8ec1b3edb7f876235bec1138e...

BUG=557624
==========

to

==========
Stop requiring NotificationAction action and title to be non-empty

Improve spec compliance. Note that
http/tests/notifications/serviceworkerregistration-document-actions-throw.html
still tests that they are set (but it's no longer possible to have an
equivalent check in NotificationDataTest, since required WebIDL
attributes are enforced at the V8 level by [1], and are bypassed when
creating IDL objects from C++).

[1]:
https://chromium.googlesource.com/chromium/src/+/dfd8ec1b3edb7f876235bec1138e...

BUG=557624

Committed: https://crrev.com/56c5368ca167742af3dfdd0bf8f879984b2ad05b
Cr-Commit-Position: refs/heads/master@{#362469}
==========

[commit-bot: I haz the power, 2015-12-01 19:19:34]

Patchset 1 (id:??) landed as
https://crrev.com/56c5368ca167742af3dfdd0bf8f879984b2ad05b
Cr-Commit-Position: refs/heads/master@{#362469}