Closed Bug 1105832 Opened 10 years ago Closed 9 years ago

Include inactive layer manager in the display list dump

Categories

(Core :: Layout, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: BenWa, Assigned: botond)

References

Details

Attachments

(2 files, 1 obsolete file)

Attached file mstange' patch (obsolete) —
Right now with the display list dump we can't match where a display item is going into the layer tree if it's drawn to an inactive layer manager.
Blocks: 1112718
Attached patch updated patchSplinter Review
Attachment #8529851 - Attachment is obsolete: true
Updated patch to integrate well into paint dumping to an HTML file. Let's get this reviewed and landed!
Bug 1105832 - Also dump inactive layer managers when display list dumping is enabled. r=mattwoodrow
Attachment #8666250 - Flags: review?(matt.woodrow)
Comment on attachment 8666250 [details]
MozReview Request: Bug 1105832 - Also dump inactive layer managers when display list dumping is enabled. r=mattwoodrow

Asking BenWa for review instead as Matt is on PTO.
Attachment #8666250 - Flags: review?(matt.woodrow) → review?(bgirard)
Comment on attachment 8666250 [details]
MozReview Request: Bug 1105832 - Also dump inactive layer managers when display list dumping is enabled. r=mattwoodrow

https://reviewboard.mozilla.org/r/20479/#review18449
Attachment #8666250 - Flags: review?(bgirard) → review+
Assignee: nobody → botond
https://hg.mozilla.org/mozilla-central/rev/76076d8dad02
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Blocks: 1462177
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: