Closed Bug 851607 Opened 9 years ago Closed 9 years ago

Make nsFlexContainerFrame::Reflow reflow its children unconditionally

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: dholbert, Assigned: dholbert)

References

Details

Attachments

(2 files)

Per discussion w/ dbaron yesterday (partly chronicled on bug 851012), our "does this frame's children need a reflow" logic (and the related mHResize / mVResize flags) are mainly appropriate for block layout, not for flexbox layout -- and we shouldn't shoehorn it to try to make it work for flexbox layout.

So: filing this bug on making flexbox-reflow eagerly reflow its children.
Summary: Make nsFlexContainerFrame::Reflow always reflow its children → Make nsFlexContainerFrame::Reflow reflow its children unconditionally
Attached patch fix v1Splinter Review
Attachment #725564 - Flags: review?
Attachment #725564 - Flags: review?
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Attachment #725566 - Flags: review?(dbaron)
Comment on attachment 725566 [details] [diff] [review]
fix v1, no-whitespace version

r=dbaron
Attachment #725566 - Flags: review?(dbaron) → review+
https://hg.mozilla.org/mozilla-central/rev/3a1ab4d11513
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
Blocks: 853946
Blocks: 785468
You need to log in before you can comment on or make changes to this bug.