See also: https://github.com/servo/servo/pull/11404

Comment on attachment 8756169 [details] [diff] [review] Part 1: Add Servo_RestyleSubtree for eagerly restyling a subtree of a document. Review of attachment 8756169 [details] [diff] [review]: ----------------------------------------------------------------- ::: layout/style/ServoStyleSet.h @@ +51,5 @@ > > void BeginUpdate(); > nsresult EndUpdate(); > > + void ForceRestyle(nsINode* aNode); I don't particularly like the use of "Force" here. To me that implies that we're throwing away cached data and restyling everything, which isn't going to be what we do here long-term. So maybe just call this RestyleSubtree?

Comment on attachment 8756170 [details] [diff] [review] Part 2: Eagerly restyle native anonymous content in stylo. Review of attachment 8756170 [details] [diff] [review]: ----------------------------------------------------------------- I don't know my way around the frame constructor well enough to say with certainty that this is the right place to do it, but seems harmless enough.

Comment on attachment 8756172 [details] [diff] [review] Part 4: Remove anonymous content styling hack in ServoStyleSet. Review of attachment 8756172 [details] [diff] [review]: ----------------------------------------------------------------- \o/

https://hg.mozilla.org/integration/mozilla-inbound/rev/a51e41ec23e2 https://hg.mozilla.org/integration/mozilla-inbound/rev/ce7e41b72c76 https://hg.mozilla.org/integration/mozilla-inbound/rev/5e15a686b21e https://hg.mozilla.org/integration/mozilla-inbound/rev/dcf1684b63b3

https://hg.mozilla.org/mozilla-central/rev/a51e41ec23e2 https://hg.mozilla.org/mozilla-central/rev/ce7e41b72c76 https://hg.mozilla.org/mozilla-central/rev/5e15a686b21e https://hg.mozilla.org/mozilla-central/rev/dcf1684b63b3