Open Bug 918064 Opened 11 years ago Updated 2 years ago

don't make a RestyleSelf call for each continuation (and similar inReParentStyleContext)

Categories

(Core :: CSS Parsing and Computation, defect)

defect

Tracking

()

People

(Reporter: dbaron, Unassigned)

References

Details

The next step following the sequence of refactorings: bug 896138, bug 898209, bug 898329, bug 898333, and bug 828312 is that we should change ElementRestyler::RestyleSelf so that it sets the style context on all continuations and block-in-inline siblings as appropriate rather than requiring a separate RestyleSelf call in each one.  We likely want to make similar changes to ReParentStyleContext.

This should remove the remainder of the things marked "TEMPORARY" from bug 898333 patch 2.
I suspect that this will bit-rot my patches for display/box:contents (in bug 907396,
bug 917313) pretty bad.  If it's just cleanup, can it wait until after that lands?
Is it possible for same-style-continuation frames to have additional style contexts?  And if so, is it possible for them to have a different number of additional style contexts?  That would make this refactoring a less straightforward than it otherwise would be.
Assignee: dbaron → nobody
Status: ASSIGNED → NEW
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.