Closed Bug 1389215 Opened 3 years ago Closed 3 years ago
Stop layerizing in response to margin animations
59 bytes, text/x-review-board-request
Comment on attachment 8895963 [details] Bug 1389215 - Don't layerize in response to margin changes. https://reviewboard.mozilla.org/r/167236/#review172492 ::: layout/painting/ActiveLayerTracker.cpp:444 (Diff revision 1) > } > return nsLayoutUtils::HasEffectiveAnimation(aFrame, aProperty); > } > > /* static */ bool > ActiveLayerTracker::IsOffsetOrMarginStyleAnimated(nsIFrame* aFrame) Could rename this function now that margins aren't taken into account.
Attachment #8895963 - Flags: review?(matt.woodrow) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/ae6148ddda6b Don't layerize in response to margin changes. r=mattwoodrow
I see an improvement in memory on AWSY: == Change summary for alert #8755 (as of August 13 2017 01:16 UTC) == Improvements: 5% Resident Memory summary macosx64-stylo opt stylo 667,672,371.49 -> 637,141,298.34 For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=8755
You need to log in before you can comment on or make changes to this bug.