Closed
Bug 1436193
Opened 7 years ago
Closed 6 years ago
Cloud ad is very slow in WebRender
Categories
(Core :: Graphics: WebRender, enhancement, P2)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
People
(Reporter: jrmuizel, Assigned: gw)
References
Details
https://jrmuizel.github.io/webrender/slow/coulds.html
It looks like we're spending a bunch of time blurring.
Reporter | ||
Comment 1•7 years ago
|
||
The url should be https://jrmuizel.github.io/webrender/slow/clouds.html
Assignee | ||
Comment 2•7 years ago
|
||
You can assign this one to me, if you like. I imagine we'll want to add image blurs to the list of items we pre-render to the texture cache, or something similar.
Reporter | ||
Updated•7 years ago
|
Assignee: nobody → gwatson
Updated•7 years ago
|
Blocks: stage-wr-trains
Priority: -- → P1
Assignee | ||
Comment 3•7 years ago
|
||
I profiled this briefly - in this case it's actually text shadows being used for the effect. We'll want to cache text-shadow blurs, just as we do for box-shadows.
Could bug 1452489 be a (long winded) duplicate of this?
Comment 5•6 years ago
|
||
We can't release this to the field, but we can let this ride to beta.
Priority: P1 → P2
Reporter | ||
Updated•6 years ago
|
Depends on: picture-caching
Assignee | ||
Comment 6•6 years ago
|
||
The initial patch for picture caching on text shadows is https://github.com/servo/webrender/pull/3270. The commit mentions:
This brings the GPU time in the cloud ad [1] from 43 ms down to
8 ms for me. This is still higher than we want, due to the very high
number of cached text shadows that get composited. This should be
resolved as we support nested picture caching.
I think it's probably OK to verify and mark this as fixed once that lands, as it makes the performance very usable. It's just not as fast as we want / expect it to be in the future.
Assignee | ||
Comment 7•6 years ago
|
||
This works really well now, with picture caching enabled by default.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•