Closed Bug 1346701 Opened 3 years ago Closed 3 years ago

Delete BorderLayer

Categories

(Core :: Graphics: WebRender, enhancement)

enhancement
Not set

Tracking

()

RESOLVED INVALID

People

(Reporter: ethlin, Assigned: ethlin)

References

Details

Attachments

(1 file)

We replace WRBorderLayer with DisplayItemLayer in bug 1346451. So there is caller to call CreateBorderLayer in the client, which means we can remove BorderLayer.
Depends on: 1346451
(In reply to Ethan Lin[:ethlin] from comment #0)
> We replace WRBorderLayer with DisplayItemLayer in bug 1346451. So there is
> caller to call CreateBorderLayer in the client, which means we can remove
> BorderLayer.

typo. There is 'no' caller to call CreateBorderLayer in the client.
Attachment #8846498 - Flags: review?(mchang)
Blocks: 1346702
Comment on attachment 8846498 [details] [diff] [review]
Remove border layer

Advanced layers still wants to use border layers, so I don't think we can delete this yet.
Attachment #8846498 - Flags: review?(mchang)
Can you please confirm comment 3? Thanks
Flags: needinfo?(dvander)
Yeah, please do not delete this or TextLayer just yet. They are still being used for experiments.
Flags: needinfo?(dvander)
I see! So let's leave it.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.