This is a regression from bug 309322. We're not reflowing the new caption...
Requesting the same blocking flags that bug 309322 has.
Comment on attachment 274251 [details] [diff] [review] Fix sr=dbaron
Attachment #274251 - Flags: superreview?(dbaron) → superreview+
Summary: [FIX]Inserting a caption into a table doesn't work → [FIXr]Inserting a caption into a table doesn't work
Comment on attachment 274251 [details] [diff] [review] Fix Make sure to mark a caption being inserted dynamically into a table as needing reflow. Very safe regression fix.
Attachment #274251 - Flags: approval1.9?
Comment on attachment 274251 [details] [diff] [review] Fix a19=dbaron
Attachment #274251 - Flags: approval1.9? → approval1.9+
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
Flags: blocking22.214.171.124? → blocking126.96.36.199?
What did we use for PresShell::FrameNeedsReflow() on the 1.8 branch? I don't know how to back-port this one.
Basically, nsTableOuterFrame::InsertFrames needs to do what nsTableOuterFrame::AppendFrames does (on branch, that uses AppendReflowCommand).
thanks, that fixed the regression. I think I'll wrap all these regressions up into a combined 1.8 branch patch for re-review.
Needed on the 1.8.0 branch only if bug 309322 lands.
Flags: blocking188.8.131.52+ → blocking184.108.40.206?
Since 309322 is marked blocking220.127.116.11+ carrying over blocking to this bug too.
Flags: blocking18.104.22.168? → blocking22.214.171.124+
You need to log in before you can comment on or make changes to this bug.