commit 2da5ac7f55e19bbab0497e9afa00fdddc5d16496
author Philip Jägenstedt <foolip@chromium.org> Tue Oct 09 08:01:34 2018
committer Commit Bot <commit-bot@chromium.org> Tue Oct 09 08:01:34 2018
tree 9eea0e83f1b502083ecc2739acd35106a9baaa3d
parent 340b56deb7102f8e32957c644f6ca3345e9c91ec

Adapt to "Disallow `test` return value" change in wpt

The change in question:
https://github.com/web-platform-tests/wpt/pull/12958

This change was done to flag possible test authoring mistakes. While
most of the updates in this CL are just adding braces arrow functions,
in set-root-scroller.html there was a genuine problem with the test.

In assert_selection.js, to avoid updating many many test using
`test(() => assert_selection(...), desc)`, change `assert_selection` to
not have a return value and add `assert_selection_and_return_sample`.

Bug: 892613
Change-Id: Idaee49e40a9d85f5dfb7124c35a0d864bdbf016f
Reviewed-on: https://chromium-review.googlesource.com/c/1268341
Reviewed-by: David Bokan <bokan@chromium.org>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#597834}