Throw when testharness::done() is called by layout test code

Throw when testharness::done() is called by layout test code

This CL wraps done() method in the global scope, which is defined by
testharness.js to prevent a test from finishing prematurely.

2 layout tests are found from this check and fixed accordingly.

BUG=708817
TESTS=
webaudio/AudioParam/audioparam-update-value-attribute.html
webaudio/DynamicsCompressor/dynamicscompressor-basic.html

Review-Url: https://codereview.chromium.org/2804103002
Cr-Commit-Position: refs/heads/master@{#462685}
Committed: https://chromium.googlesource.com/chromium/src/+/897d20df0777854e91b4afab0923e917cd06640a

[hongchan, 2017-04-06 19:38:23]

hongchan@chromium.org changed reviewers:
+ rtoy@chromium.org

[hongchan, 2017-04-06 19:38:24]

PTAL. Confirmed all tests are passing locally.

[Raymond Toy, 2017-04-06 20:02:05]

lgtm

[hongchan, 2017-04-06 20:33:38]

The CQ bit was checked by hongchan@chromium.org

[commit-bot: I haz the power, 2017-04-06 20:34:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-06 23:40:54]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491510818795730, "parent_rev":
"b2b99a3b809f5ecbad2c3c9106dda9835e39a247", "commit_rev":
"897d20df0777854e91b4afab0923e917cd06640a"}

[commit-bot: I haz the power, 2017-04-06 23:41:44]

Description was changed from

==========
Throw when testharness::done() is called by layout test code

This CL wraps done() method in the global scope, which is defined by
testharness.js to prevent a test from finishing prematurely.

2 layout tests are found from this check and fixed accordingly.

BUG=708817
TESTS=
webaudio/AudioParam/audioparam-update-value-attribute.html
webaudio/DynamicsCompressor/dynamicscompressor-basic.html
==========

to

==========
Throw when testharness::done() is called by layout test code

This CL wraps done() method in the global scope, which is defined by
testharness.js to prevent a test from finishing prematurely.

2 layout tests are found from this check and fixed accordingly.

BUG=708817
TESTS=
webaudio/AudioParam/audioparam-update-value-attribute.html
webaudio/DynamicsCompressor/dynamicscompressor-basic.html

Review-Url: https://codereview.chromium.org/2804103002
Cr-Commit-Position: refs/heads/master@{#462685}
Committed:
https://chromium.googlesource.com/chromium/src/+/897d20df0777854e91b4afab0923...
==========

[commit-bot: I haz the power, 2017-04-06 23:41:45]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/897d20df0777854e91b4afab0923...