Closed
Bug 1332561
Opened 8 years ago
Closed 8 years ago
Enable async video frame update in WebRenderImageLayer
Categories
(Core :: Graphics: WebRender, defect)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla54
People
(Reporter: sotaro, Assigned: sotaro)
References
Details
Attachments
(1 file, 1 obsolete file)
5.30 KB,
patch
|
nical
:
review+
|
Details | Diff | Splinter Review |
WebRenderImageLayer does not use async image rendering. All video update is done via main thread. We need to enable it.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → sotaro.ikeda.g
Assignee | ||
Comment 1•8 years ago
|
||
Assignee | ||
Updated•8 years ago
|
Summary: Enable async video frame update → Enable async video frame update in WebRenderImageLayer
Assignee | ||
Comment 3•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=e7975a1e84c79bdcb152132415bd90de24a76b92
Pushed by sikeda@mozilla.com: https://hg.mozilla.org/projects/graphics/rev/585ed02acd43 Enable async video frame update in WebRenderImageLayer r=nical?
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•8 years ago
|
Attachment #8828704 -
Flags: review?(nical.bugzilla)
Comment 5•8 years ago
|
||
Comment on attachment 8828704 [details] [diff] [review] patch - Enable async video frame update in WebRenderImageLayer Review of attachment 8828704 [details] [diff] [review]: ----------------------------------------------------------------- ::: gfx/layers/wr/WebRenderImageLayer.cpp @@ +90,4 @@ > return; > } > > + MOZ_ASSERT(GetImageClientType() != CompositableType::UNKNOWN); nit: maybe not necessary to assert this just after a a branch that trivially prevents it.
Attachment #8828704 -
Flags: review?(nical.bugzilla) → review+
Updated•8 years ago
|
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•