Bug 1582809 [wpt PR 19192] - [testharness.js] remove superfluous `test.done()` call, a=testonly
authorPhilip Jägenstedt <philip@foolip.org>
Mon, 23 Sep 2019 12:25:32 +0000
changeset 494703 8e0f065609cf155c742bc1050366f0c60cd3d1d9
parent 494702 a2528ae9e8aae142ba6df872ce8d07d6011f6647
child 494704 e52193058ac02f7e7c7d1e8e9986600055493b1d
push id114126
push userwptsync@mozilla.com
push dateTue, 24 Sep 2019 19:34:44 +0000
treeherdermozilla-inbound@068d8731f1c4 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstestonly
bugs1582809, 19192
milestone71.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1582809 [wpt PR 19192] - [testharness.js] remove superfluous `test.done()` call, a=testonly Automatic update from web-platform-tests [testharness.js] remove superfluous `test.done()` call (#19192) -- wpt-commits: 5ec2527658fe97003e0691decb44a121852d715c wpt-pr: 19192
testing/web-platform/tests/resources/testharness.js
--- a/testing/web-platform/tests/resources/testharness.js
+++ b/testing/web-platform/tests/resources/testharness.js
@@ -3673,19 +3673,16 @@ policies and contribution forms [3].
 
             if (tests.file_is_test) {
                 var test = tests.tests[0];
                 if (test.phase >= test.phases.HAS_RESULT) {
                     return;
                 }
                 test.set_status(test.FAIL, e.message, stack);
                 test.phase = test.phases.HAS_RESULT;
-                // The following function invocation is superfluous.
-                // TODO: Remove.
-                test.done();
             } else if (!tests.allow_uncaught_exception) {
                 tests.status.status = tests.status.ERROR;
                 tests.status.message = e.message;
                 tests.status.stack = stack;
             }
             done();
         };