Closed Bug 1885437 Opened 1 year ago Closed 1 year ago

DAMP tests doesn't wait for styleeditor to fetch the stylesheet content on page reload

Categories

(DevTools :: Style Editor, task)

task

Tracking

(firefox126 fixed)

RESOLVED FIXED
126 Branch
Tracking Status
firefox126 --- fixed

People

(Reporter: ochameau, Assigned: ochameau)

References

Details

Attachments

(1 file)

This ended up skewing tests when applying the patch queue from bug 1713093.

This will regress, expectedly, the reload tests of the styleeditor.

The test wasn't really measuring anything meaningful and resolve abruptly.

Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED
Pushed by apoirot@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/90c7bb425e92 [devtools] Wait for style editor to fetch the source content in DAMP tests when reloading pages. r=devtools-reviewers,perftest-reviewers,nchevobbe,sparky
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 126 Branch

== Change summary for alert #41938 (as of Thu, 21 Mar 2024 10:03:21 GMT) ==

Regressions:

Ratio Test Platform Options Absolute values (old vs new)
69% damp custom.styleeditor.reload.DAMP windows10-64-shippable-qr e10s fission stylo webrender 364.15 -> 617.02
68% damp custom.styleeditor.reload.DAMP macosx1015-64-shippable-qr e10s fission stylo webrender-sw 301.41 -> 506.00
63% damp custom.styleeditor.reload.DAMP linux1804-64-shippable-qr e10s fission stylo webrender 433.99 -> 705.32
62% damp custom.styleeditor.reload.DAMP windows10-64-shippable-qr e10s fission stylo webrender-sw 360.16 -> 583.17
58% damp custom.styleeditor.reload.DAMP macosx1015-64-shippable-qr e10s fission stylo webrender 313.65 -> 496.01
57% damp custom.styleeditor.reload.DAMP linux1804-64-shippable-qr e10s fission stylo webrender-sw 428.70 -> 674.00
24% damp custom.styleeditor.close.DAMP windows10-64-shippable-qr e10s fission stylo webrender-sw 10.83 -> 13.43
23% damp complicated.styleeditor.close.DAMP linux1804-64-shippable-qr e10s fission stylo webrender-sw 11.61 -> 14.33
22% damp custom.styleeditor.close.DAMP windows10-64-shippable-qr e10s fission stylo webrender 10.66 -> 13.00
20% damp complicated.styleeditor.close.DAMP windows10-64-shippable-qr e10s fission stylo webrender-sw 12.18 -> 14.68
... ... ... ... ...
10% damp complicated.styleeditor.reload.DAMP linux1804-64-shippable-qr e10s fission stylo webrender-sw 1,137.59 -> 1,251.64
10% damp complicated.styleeditor.reload.DAMP linux1804-64-shippable-qr e10s fission stylo webrender 1,188.04 -> 1,301.40
7% damp complicated.styleeditor.reload.DAMP windows10-64-shippable-qr e10s fission stylo webrender-sw 1,200.56 -> 1,290.21
7% damp custom.netmonitor.reload.DAMP linux1804-64-shippable-qr e10s fission stylo webrender 341.23 -> 365.47
7% damp complicated.styleeditor.reload.DAMP windows10-64-shippable-qr e10s fission stylo webrender 1,208.78 -> 1,288.14

For up to date results, see: https://treeherder.mozilla.org/perfherder/alerts?id=41938

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: