Closed Bug 1397813 Opened 7 years ago Closed 7 years ago

stylo: we're not memory-reporting some ComputedValues for text

Categories

(Core :: CSS Parsing and Computation, enhancement, P3)

53 Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

See bug 1397380 comment 8 item 3.

We have text ComputedValues that we're not reporting.

Maybe we should be reporting the stuff living in mNextInheritingAnonBoxStyle and mNextLazyPseudoStyle in ServoStyleContext; right now it looks like it could be non-reported if it's not reachable from any frames.
Blocks: 1397380
This eliminates the unreported text style contexts on the HTML spec for me
Attachment #8905611 - Flags: review?(n.nethercote)
Comment on attachment 8905611 [details] [diff] [review]
Make sure to memory-report the ServoStyleContexts for anon boxes and lazy pseudos that we cache on the parent ServoStyleContext

Review of attachment 8905611 [details] [diff] [review]:
-----------------------------------------------------------------

Looks reasonable. Did you verify with DMD?
Attachment #8905611 - Flags: review?(n.nethercote) → review+
Yes, I did.  That's what the "This eliminates ..." comment was about.
Priority: -- → P3
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f0fdf712c724
Make sure to memory-report the ServoStyleContexts for anon boxes and lazy pseudos that we cache on the parent ServoStyleContext.  r=njn
https://hg.mozilla.org/mozilla-central/rev/f0fdf712c724
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.