1685345 - Consolidate WR texture allocation code into a module

Status: RESOLVED FIXED

(Core :: Graphics: WebRender, task)

Description

[Dzmitry Malyshau [:kvark]]

We have the following modules in the root:

• texture_cache.rs
• slab_allocator.rs
• atlas_allocator.rs
• guillotine_allocator.rs

I believe all those are close in semantics and should all be moved into a module. Something like this:

• src/texture_cache/mod.rs
• src/texture_cache/allocator/slab.rs
• src/texture_cache/allocator/atlas.rs
• src/texture_cache/allocator/guillotine.rs

Comment 1

[Dzmitry Malyshau [:kvark]]

Attachment: Move WR texture packing into a module

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b8e9cf21898a
Move WR texture packing into a module r=nical

Comment 3

[Dorel Luca [:dluca]]

Backed out changeset b8e9cf21898a (bug 1685345) for WebRender build bustages. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer?job_id=326081973&repo=autoland&lineNumber=30272

https://treeherder.mozilla.org/logviewer?job_id=326081969&repo=autoland&lineNumber=13041

Push with failures:
https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&revision=b8e9cf21898ae7b100f99d88901363a9bd886cac

Backout:
https://hg.mozilla.org/integration/autoland/rev/9070ce19efe09f08b427693061abd6ee3010bc85

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/04d20091236f
Move WR texture packing into a module r=nical

Comment 5

[Dzmitry Malyshau [:kvark]]

Was a harmless accidental WR warning due to a slight change in includes together with the logic that uses them (in an unrelated change). Re-landed now.

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/04d20091236f