Closed Bug 1713350 Opened 3 years ago Closed 3 years ago

SVG rendering hangs up in GDocs, causing significant performance issues.

Categories

(Core :: Graphics: WebRender, defect, P2)

defect

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox88 --- unaffected
firefox89 --- unaffected
firefox90 --- fixed

People

(Reporter: mhoye, Assigned: aosmond)

References

(Regression)

Details

(Keywords: perf-alert, regression)

Attachments

(1 file)

I'm experiencing a significant perf hit in GDocs, a type-then-wait-for-the letters-to-appear kind of thing. The performance profile is here:

https://share.firefox.dev/2SCxWI9

We are apparently blocking a the responsible thread for 650 of every 700 milliseconds which frankly seems like a lot. Per Slack, https://bugzilla.mozilla.org/show_bug.cgi?id=1704792 may be related.

Does setting image.svg.blob-image to false and restarting help?

Flags: needinfo?(mhoye)

Never mind, I can see it in the trace.

Flags: needinfo?(mhoye)
Assignee: nobody → aosmond
Status: NEW → ASSIGNED
Pushed by aosmond@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f845b3a52e7d Ensure we reset the dirty flag for SVG blob recordings during moves. r=jrmuizel
Keywords: regression
Regressed by: 1704792
See Also: 1704792
Has Regression Range: --- → yes
Severity: -- → S2
Priority: -- → P2
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch

Can you retest and confirm it is fixed, or provide a trace if not? Thanks!

Flags: needinfo?(mhoye)

I have not been able to reproduce this in the last few days, let's call this fixed.

Flags: needinfo?(mhoye)

(In reply to Pulsebot from comment #4)

Pushed by aosmond@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f845b3a52e7d
Ensure we reset the dirty flag for SVG blob recordings during moves.
r=jrmuizel

== Change summary for alert #30257 (as of Sun, 30 May 2021 19:06:15 GMT) ==

Improvements:

Ratio Suite Test Platform Options Absolute values (old vs new)
19% tart (docs) macosx1015-64-shippable-qr e10s stylo webrender-sw 2.58 -> 2.09
17% tart (docs) windows10-64-shippable-qr e10s stylo webrender-sw 3.33 -> 2.77
16% tart (docs) macosx1015-64-shippable-qr e10s stylo webrender 2.53 -> 2.13
16% tart (docs) windows10-64-shippable-qr e10s stylo webrender 3.33 -> 2.81
15% tart (docs) macosx1014-64-shippable-qr e10s stylo webrender 4.98 -> 4.22
7% tart (docs) linux1804-64-shippable-qr e10s stylo webrender 4.10 -> 3.82

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

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

Attachment

General

Created:
Updated:
Size: