Closed Bug 1709272 Opened 7 months ago Closed 6 months ago

30.51 - 18.79% google-mail LastVisualChange / youtube SpeedIndex (Windows) regression on Mon April 26 2021

Categories

(Firefox :: Theme, defect)

defect

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox-esr78 --- unaffected
firefox88 --- unaffected
firefox89 --- wontfix
firefox90 --- wontfix

People

(Reporter: Bebe, Unassigned)

References

(Regression)

Details

(Keywords: perf, perf-alert, regression)

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

Regressions:

Ratio Suite Test Platform Options Absolute values (old vs new)
31% google-mail LastVisualChange windows10-64-shippable warm 1,846.67 -> 2,410.00
19% youtube SpeedIndex windows10-64-shippable-qr warm webrender 820.54 -> 974.75

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 offending patch(es) will be backed out in accordance with our regression policy.

For more information on performance sheriffing please see our FAQ.

Flags: needinfo?(jaws)

Set release status flags based on info from the regressing bug 1703957

Beta alert:

== Change summary for alert #29925 (as of Fri, 30 Apr 2021 15:00:16 GMT) ==

Regressions:

Ratio Suite Test Platform Options Absolute values (old vs new)
20% youtube SpeedIndex windows10-64-shippable-qr warm webrender 807.77 -> 971.08

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

Jaws, is that a perf regression we should fix for 89? Thanks

The connection here to the regressing bug doesn't make sense to me. The patch there just changed some colors in the theme and should have zero effect on performance. Can we reevaluate what bug may have caused this regression?

Flags: needinfo?(jaws) → needinfo?(fstrugariu)

this bug got fixed by alert: https://treeherder.mozilla.org/perfherder/alerts?id=29967
https://bugzilla.mozilla.org/show_bug.cgi?id=1707949#c9

as browsertime visual metrics compare actual videos of the browser it might catch chrome changes

Flags: needinfo?(fstrugariu)

Does comment 5 mean we can close this bug as fixed?

Flags: needinfo?(fstrugariu)

Yes. closing this as won't fix

Status: NEW → RESOLVED
Closed: 6 months ago
Flags: needinfo?(fstrugariu)
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.