1724846 - Split DL spatial tree to separate payload

Status: RESOLVED FIXED

(Core :: Graphics: WebRender, task)

Comment 1

[Glenn Watson [:gw]]

Attachment: Bug 1724846 - Split DL spatial tree to separate payload

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.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5f2aff372a24
Split DL spatial tree to separate payload r=gfx-reviewers,nical

Comment 3

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/5f2aff372a24

Comment 4

[Narcis Beleuzu [:NarcisB]]

Backed out as per gw`s request
Backout link: https://hg.mozilla.org/integration/autoland/rev/66e7f6f416b35c06cc33f540e08a59b42d2de99d

Comment 5

[Atila Butkovits]

Backout merged to central: https://hg.mozilla.org/mozilla-central/rev/66e7f6f416b35c06cc33f540e08a59b42d2de99d

Comment 6

[Glenn Watson [:gw]]

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

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d93e8a61d529
Split DL spatial tree to separate payload r=gfx-reviewers,nical

Comment 8

[Iulian Moraru]

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

Push with failures

Failure log R1
Failure log R2

Backout link

Comment 9

[Timothy Nikkel (:tnikkel)]

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.

Comment 10

[Glenn Watson [:gw]]

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.

Comment 11

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/1d332a5ff4f9

Comment 12

[Alexandru Michis [:malexandru]]

Backed out changeset 1d332a5ff4f9 (Bug 1724846) for causing Bug 1727715.
Backout link: https://hg.mozilla.org/integration/autoland/rev/d4b71143f9a7937e24bf57f9fb56f1bf414b55c0

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/438c4a4cf4b3
Split DL spatial tree to separate payload r=gfx-reviewers,nical

Comment 14

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/438c4a4cf4b3