Closed Bug 1724846 Opened 2 months ago Closed 2 months ago

Split DL spatial tree to separate payload

Categories

(Core :: Graphics: WebRender, task)

task

Tracking

()

RESOLVED FIXED
93 Branch
Tracking Status
firefox93 --- fixed

People

(Reporter: gw, Assigned: gw)

References

Details

Attachments

(1 file)

No description provided.

This will allow experimenting with different representations of
the spatial tree (such as interning and/or providing stable
indices during display list building). It may also simplify
future changes to the public API to expose the spatial tree
directly.

As part of these changes, refactor how the debug representation
for the capture format is (de)serialized, to make it simpler to
add different payload vector types in future.

Assignee: nobody → gwatson
Status: NEW → ASSIGNED
Pushed by gwatson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5f2aff372a24
Split DL spatial tree to separate payload r=gfx-reviewers,nical
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 93 Branch → ---
Flags: needinfo?(gwatson)

Thanks, updated the patch and sent for re-review.

Flags: needinfo?(gwatson)
Pushed by gwatson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d93e8a61d529
Split DL spatial tree to separate payload r=gfx-reviewers,nical

Backed out for causing reftest failures on downscale-moz-icon-1.html.

Push with failures

Failure log R1
Failure log R2

Backout link

Flags: needinfo?(gwatson)

As the author of that test I'm fine with adding fuzz to it if you don't think it's an issue with your patch.

Thanks Timothy - I've added the few extra pixels fuzz for that, it seems unrelated to the patch itself. Will do another try run then try to re-land.

Flags: needinfo?(gwatson)
Status: REOPENED → RESOLVED
Closed: 2 months ago2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch
Regressions: 1727715
Regressions: 1727795
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 93 Branch → ---
Pushed by gwatson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/438c4a4cf4b3
Split DL spatial tree to separate payload r=gfx-reviewers,nical
Status: REOPENED → RESOLVED
Closed: 2 months ago2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch
You need to log in before you can comment on or make changes to this bug.