tree 0e58cbd522f6fc562f220ed6d2576f90acd41362
parent 21369c1a88085d4804d520887978392d19c8c656
author Mike Pennisi <mike@mikepennisi.com> 1530286078 -0400
committer Ms2ger <Ms2ger@gmail.com> 1530541865 +0200

[fullscreen] Do not return value from cleanup fns

Today, the return value of functions provided to the global
`add_cleanup` function has no effect on the behavior of the test runner.
An upcoming feature addition to testharness.js will cause the return
value to influence test results [1].

Despite this, some existing tests have already been authored to return a
value: the result of `document.exitFullScreen`. Although this is
expected to be a Promise in conforming implementations, some browsers do
not yet implement this functionality.

To allow the new test harness feature to land without introducing
harness errors, refactor existing tests to omit a return value.

Additionally, use `Promise.prototype.catch` to avoid race conditions
resulting from unhandled Promise rejections (which trigger a harness
error in testharness.js today).

[1] https://github.com/web-platform-tests/wpt/issues/6075