RESOLVED FIXED in Firefox 63

Status

()

enhancement
P2
normal
RESOLVED FIXED
Last year
10 months ago

People

(Reporter: jrmuizel, Assigned: mstange)

Tracking

(Blocks 2 bugs, {perf})

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox58 unaffected, firefox59 unaffected, firefox60 disabled, firefox63 fixed)

Details

()

Attachments

(1 attachment)

https://perfht.ml/2ntifR9

We're doing a bunch of blur. I'm not sure why it's so slow.

Things that can be done to fix it:
- Figure out why it's so slow (it shouldn't be as bad as it is)
- The blur should be handled by WebRender because it's outside of the svg
- Reduce the tile size of blob image so that we parallel paint it.
Assignee

Comment 2

10 months ago
(In reply to Jeff Muizelaar [:jrmuizel] from comment #0)
> https://perfht.ml/2ntifR9
> 
> We're doing a bunch of blur. I'm not sure why it's so slow.
> 
> Things that can be done to fix it:
> - Figure out why it's so slow (it shouldn't be as bad as it is)

I think it's just a gigantic blur radius and we don't do anything smart (like downscale-before-blur) to address that.

> - The blur should be handled by WebRender because it's outside of the svg

I've attached a patch that fixes this part.

> - Reduce the tile size of blob image so that we parallel paint it.

We're still not using tiles for blob images effectively, so this still needs to be fixed.
Reporter

Comment 3

10 months ago
Comment on attachment 9003292 [details]
Bug 1435045 - Don't make CSS filters on the <svg> element use fallback. r=jrmuizel

Jeff Muizelaar [:jrmuizel] has approved the revision.
Attachment #9003292 - Flags: review+

Comment 4

10 months ago
Pushed by mstange@themasta.com:
https://hg.mozilla.org/integration/autoland/rev/d77ca862636d
Don't make CSS filters on the <svg> element use fallback. r=jrmuizel

Comment 5

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d77ca862636d
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Assignee: nobody → mstange
You need to log in before you can comment on or make changes to this bug.