1527026 - Add WebRender support for container item hit test information

Status: RESOLVED INVALID

(Core :: Graphics: WebRender, enhancement, P3)

Description

[Miko Mynttinen]

Bug 1503046 added support for carrying hit test information with container items, and removed the need to create additional nsDisplayCompositorHitTestInfo items.

This feature is currently disabled with WebRender1, because WebRenderCommandBuilder does not know how to read the information.

Comment 1

[Miko Mynttinen]

After implementing "display items with type" iterator for WebRenderCommandBuilder, and moving stacking context hit test information from nsDisplayCompositorHitTestInfo items to container items, the results of display list mutate talos test look good1 with around 12% improvement.
These benchmarks do not include the cost of processing the hit test info, since that is not yet implemented, but I am not expecting that to change results by more than a percent or two.

Comment 2

[Miko Mynttinen]

Attachment: wr-flatten-dl.patch

WIP patch to change WebRenderCommandBuilder display list iterator

Comment 3

[Jeff Muizelaar [:jrmuizel]]

Miko, what's the plan for this bug?

Comment 4

[Miko Mynttinen]

I think it makes sense to fix the regressions of bug 1534549 first. That bug can be fixed by creating more hit test items (the parent hit test reuse logic is broken), hopefully after we reduce the cost of memory allocations for display items (bug 1539597 and bug 1554503).
After that, we can profile and estimate the performance improvement of creating fewer hit test items.

Comment 5

[Miko Mynttinen]

Obsolete after changes in bug 1534549.