Closed Bug 1353627 Opened 3 years ago Closed 3 years ago

Make GetNextExternalImageId() as to use GetNextResourceId()

Categories

(Core :: Graphics: WebRender, enhancement)

enhancement
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.
Blocks: 1345054
Another way to address the problem is use name space that is not used by LayersId.
Assignee: nobody → sotaro.ikeda.g
(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.
Summary: Split PipelineId and LayersId → Make GetNextExternalImageId() as to use GetNextResourceId()
With attachment 8854718 [details] [diff] [review], ExternalImageId do not conflict with PipelineId, since PipelineId.mNamespace is 0 and ExternalImageId's mNamespace is not 0.
With attachment 8854718 [details] [diff] [review], ExternalImageId could be used for PipelineId of Async video.
Attachment #8854718 - Flags: review?(jmuizelaar)
Attachment #8854718 - Flags: review?(jmuizelaar)
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.