Closed Bug 1353948 Opened 7 years ago Closed 7 years ago

stylo: Add SizeOfIncludingThis() methods to ServoStyle{Set,Sheet,SheetInner}

Categories

(Core :: Layout, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: n.nethercote, Assigned: n.nethercote)

References

Details

Attachments

(1 file)

This fills things in a bit more on the Gecko side.
Blocks: 1281964
Summary: Add SizeOfIncludingThis() methods to ServoStyle{Set,Sheet,SheetInner} → stylo: Add SizeOfIncludingThis() methods to ServoStyle{Set,Sheet,SheetInner}
Comment on attachment 8855132 [details] [diff] [review]
Add SizeOfIncludingThis() methods to ServoStyle{Set,Sheet,SheetInner}

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

::: layout/style/CSSStyleSheet.cpp
@@ +190,5 @@
>  
>      // Measurement of the following members may be added later if DMD finds it
>      // is worthwhile:
>      // - s->mRuleCollection
> +    // - s->mScopeElement

This one is non-owning.
Attachment #8855132 - Flags: review?(cam) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/27887cedb0d9d38e6d7883b664897d605b8868f3
Bug 1353948 - Add SizeOfIncludingThis() methods to ServoStyle{Set,Sheet,SheetInner}. r=heycam.
https://hg.mozilla.org/mozilla-central/rev/27887cedb0d9
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: