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

RESOLVED INACTIVE

Status

()

Core
CSS Parsing and Computation
RESOLVED INACTIVE
5 years ago
a day ago

People

(Reporter: David Baron, Assigned: David Baron)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

5 years ago
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.

Comment 3

a day ago
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: ASSIGNED → RESOLVED
Last Resolved: a day ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.