1822486 - [wpt-sync] Sync PR 39007 - [wptrunner] Fix test ID reported in `testharnessreport-content-shell.js`

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 39007 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/39007
Details from upstream follow.

Jonathan Lee <[e-mail]> wrote:

[wptrunner] Fix test ID reported in testharnessreport-content-shell.js

Currently, the content_shell reporter uses HTMLAnchorElement to parse location.href to derive the test ID, which is plumbed through content_shell's internal testRunner API and into the wptrunner driver.

For an unknown reason, the URL parts of the HTMLAnchorElement can be undefined, leading to crbug.com/1418753. It's not clear if there's a good reason why we use <a> for parsing the URL eagerly, other than copying other testharnessreport-*.js. Indeed, servodriver just uses window.location.* 1 directly on completion, which also seems to fix the linked crbug.

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=7548d583aac680f62f435fa84166939190bada74

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c4906009afcd
[wpt PR 39007] - [wptrunner] Fix test ID reported in `testharnessreport-content-shell.js`, a=testonly

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 5

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/c4906009afcd