Closed Bug 804893 Opened 8 years ago Closed 7 years ago

Refactor shadow layers

Categories

(Core :: Graphics: Layers, defect)

15 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: nrc, Assigned: nrc)

References

(Depends on 1 open bug)

Details

Refactor the layers system so that all shadow layers backends use a single layer manager and a backend-specific compositor. Start by refactoring the OGL layers backend into a CompositorOGL for OMTC and LayerManagerOGL for on-MTC.
Work is happening on the graphics branch.
Blocks: 804894
Blocks: omtcd3d9, 756606
No longer blocks: 804894
Currently TODO before a first, imperfect landing: Tiled Thebes layers, fix canvas layers, a few minor OGL tidy ups.

Once we've landed that, need to change (simplify) the API for Texture clients, work on a better API for texture client factories, implement a PTexture IPDL protocol for direct communication between texture clients and hosts (remove Update() from TextureHost).
Blocks: 804905
Depends on: 825944
Depends on: 825950
Depends on: 825948, 825937, 825934, 825932, 825931
Depends on: 834876
Depends on: 837012
Depends on: 838854
Depends on: 839237
Depends on: 839243
Depends on: 839538
Depends on: 840979
Depends on: 842518
Depends on: 843663
Depends on: 843686
Depends on: 844928
Depends on: 845326
Depends on: 847914
Depends on: 854484
Blocks: 830750
There are many box-shadow (and text-shadow) bugs that may be touched by the refactoring. Should I list them here (I cannot add them as depending or dependant, though)?
No longer depends on: 825944
Depends on: 856079
(In reply to Florian Bender from comment #3)
> There are many box-shadow (and text-shadow) bugs that may be touched by the
> refactoring. Should I list them here (I cannot add them as depending or
> dependant, though)?

How will they be affected? 'Shadow' in the context of this bug means the copy of the layer tree present on the compositor thread, not shadows as in box-shadow.
Oh, sorry, that's indeed something very different! Now all this stuff makes a lot more sense to me …
Depends on: 858914
Depends on: 859853
Depends on: 861246
Depends on: 861377
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.