Closed
Bug 1353627
Opened 3 years ago
Closed 3 years ago
Make GetNextExternalImageId() as to use GetNextResourceId()
Categories
(Core :: Graphics: WebRender, enhancement)
Core
Graphics: WebRender
Not set
Tracking
()
RESOLVED
DUPLICATE
of bug 1358014
People
(Reporter: sotaro, Assigned: sotaro)
References
Details
Attachments
(1 file)
This bug is preparation of Bug 1345054. To support video pipeline, WebRenderImageLayer needs to allocate pipeline id. But current WebRenderBridgeParent uses LayersId as PipelineId and it is allocated by GPUProcessManager. To support video pipeline, we need to split PipelineId and LayersId.
Assignee | ||
Comment 1•3 years ago
|
||
Another way to address the problem is use name space that is not used by LayersId.
Assignee: nobody → sotaro.ikeda.g
Assignee | ||
Comment 2•3 years ago
|
||
(In reply to Sotaro Ikeda [:sotaro] from comment #1) > Another way to address the problem is use name space that is not used by > LayersId. It seems better approach, since video pipeline might be removed when WebRender supports partial list update.
Assignee | ||
Updated•3 years ago
|
Summary: Split PipelineId and LayersId → Make GetNextExternalImageId() as to use GetNextResourceId()
Assignee | ||
Comment 3•3 years ago
|
||
Assignee | ||
Comment 4•3 years ago
|
||
With attachment 8854718 [details] [diff] [review], ExternalImageId do not conflict with PipelineId, since PipelineId.mNamespace is 0 and ExternalImageId's mNamespace is not 0.
Assignee | ||
Comment 5•3 years ago
|
||
With attachment 8854718 [details] [diff] [review], ExternalImageId could be used for PipelineId of Async video.
Assignee | ||
Comment 6•3 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=f19da5797f57f3de480ca263dc6b73ae9039f030
Assignee | ||
Updated•3 years ago
|
Attachment #8854718 -
Flags: review?(jmuizelaar)
Assignee | ||
Updated•3 years ago
|
Attachment #8854718 -
Flags: review?(jmuizelaar)
Assignee | ||
Comment 7•3 years ago
|
||
This bug is replaced by bug 1358014.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1358014
You need to log in
before you can comment on or make changes to this bug.
Description
•