1891213 - [wpt-sync] Sync PR 45002 - Build summary table without innerHTML

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 45002 into mozilla-central (this bug is closed when the sync is complete).

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

Daniel Vogelheim <[e-mail]> wrote:

Build summary table without innerHTML

The summary table is currently built up as a (potentially rather long) text string and then passed to innerHTML. Let's instead use DOM methods to build it up.

This fixes the issue in https://github.com/web-platform-tests/wpt/issues/44352. It might also improve readability.

Results generation was measured for the test with the largest result table (html/dom/reflection-text.text), which was around 800ms. The comment warning about "tens of seconds of execution time" from textContent calls alone seems to no longer be accurate.

Comment 1

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

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

Comment 2

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

The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cb3f80b0a295
[wpt PR 45002] - Build summary table without innerHTML, a=testonly

Comment 4

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

Test result changes from PR not available.

Comment 5

[Cristina Horotan [:chorotan]]

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