1631120 - [wpt-sync] Sync PR 23079 - [performance-timeline] simplify idlharness.js test signficantly

Status: RESOLVED FIXED

(Core :: Performance, task, P4)

Description

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

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

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

Philip Jägenstedt <[e-mail]> wrote:

[performance-timeline] simplify idlharness.js test signficantly

All of the complexity here was due to using a PerformanceMark to test
some of the aspects of PerformanceEntry which can only be tested with an
instance, and PerformanceEntry is an abstract interface with no
instances.

However, the downside is that this ends up testing more than should be
tested for Performance Timeline. This is not a good tradeoff.

Comment 1

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=e4b9c2a7eadaa2af178bf3892c279313bb53cde5

Comment 2

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

CI Results

Ran 13 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 63 tests

Status Summary

Firefox

OK : 4
PASS : 224[GitHub] 283[Gecko-android-em-7.0-x86_64-debug-geckoview, Gecko-android-em-7.0-x86_64-opt-geckoview, Gecko-linux1804-64-asan-opt, Gecko-linux1804-64-debug, Gecko-linux1804-64-opt, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows10-64-debug, Gecko-windows10-64-opt, Gecko-windows10-64-qr-debug, Gecko-windows10-64-qr-opt, Gecko-windows7-32-debug, Gecko-windows7-32-opt]

Chrome

OK : 4
PASS : 224

Safari

OK : 3
PASS : 159
FAIL : 9
ERROR: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/51992ac4f82d
[wpt PR 23079] - [performance-timeline] simplify idlharness.js test signficantly, a=testonly

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a68bcec66a52
[wpt PR 23079] - [performance-timeline] simplify idlharness.js test signficantly, a=testonly

Comment 5

[Bogdan Tara[:bogdan_tara | bogdant]]

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