1921726 - 10.36 - 5.34% damp custom.netmonitor.exportHar / damp custom.netmonitor.exportHar (OSX, Windows) regression on Tue September 24 2024

Status: RESOLVED INVALID

(DevTools :: Netmonitor, defect)

Description

[Treeherder Bug Filer]

Perfherder has detected a devtools performance regression from push bdee0d8e5018b51c89bb9ca459c21796bcffbc2d. As author of one of the patches included in that push, we need your help to address this regression.

Regressions:

Ratio	Test	Platform	Options	Absolute values (old vs new)
10%	damp custom.netmonitor.exportHar	windows11-64-shippable-qr	e10s fission stylo webrender	234.49 -> 258.78
5%	damp custom.netmonitor.exportHar	macosx1015-64-shippable-qr	e10s fission stylo webrender	514.97 -> 542.48

Details of the alert can be found in the alert summary, including links to graphs and comparisons for each of the affected tests. Please follow our guide to handling regression bugs and let us know your plans within 3 business days, or the patch(es) may be backed out in accordance with our regression policy.

If you need the profiling jobs you can trigger them yourself from treeherder job view or ask a sheriff to do that for you.

You can run all of these tests on try with ./mach try perf --alert 2235

The following documentation link provides more information about this command.

For more information on performance sheriffing please see our FAQ.

If you have any questions, please do not hesitate to reach out to [e-mail].

Comment 1

[Tooru Fujisawa [:arai]]

The "custom" test hits the cached CSS case 99 times for each load, which mean there were 99 missing requests in the network monitor before the patch because caches CSS loads weren't reported, and the patch adds them as a fix.
So the additional work is expected and the change to the score is also expected.

Comment 2

[Nicolas Chevobbe [:nchevobbe]]

Thanks for the explanation arai, looks like we can close this bug then 👍