Changeset 14899 for trunk/WebCore/rendering/RenderBlock.h

Timestamp:

06/17/06 08:54:58 (3 years ago)

Author:

adele

Message:

LayoutTests:

Reviewed by Hyatt.

• test and updated results for http://bugzilla.opendarwin.org/show_bug.cgi?id=7395 Table not properly re-flowed when floated div removed from layout

• css2.1/t0905-c5525-fltmult-00-d-g-expected.checksum: Updated.
• css2.1/t0905-c5525-fltmult-00-d-g-expected.png: Updated.
• css2.1/t0905-c5525-fltmult-00-d-g-expected.txt: Updated.
• fast/block/float/table-relayout-expected.checksum: Added.
• fast/block/float/table-relayout-expected.png: Added.
• fast/block/float/table-relayout-expected.txt: Added.
• fast/block/float/table-relayout.html: Added.

WebCore:

Reviewed by Hyatt.

• fix http://bugzilla.opendarwin.org/show_bug.cgi?id=7395 Table not properly re-flowed when floated div removed from layout

Test: fast/block/float/table-relayout.html

• rendering/RenderBlock.cpp: (WebCore::RenderBlock::markAllDescendantsWithFloatsForLayout): Added marking of children that use lineWidth.
• rendering/RenderBlock.h: Added a FIXME.

Files:

• trunk/WebCore/rendering/RenderBlock.h (modified) (1 diff)

trunk/WebCore/rendering/RenderBlock.h

@@ -168 +168 @@
     void markPositionedObjectsForLayout();
 
+    // FIXME: containsFloats() should not return true if the floating objects list
+    // is empty. However, layoutInlineChildren() relies on the current behavior.
+    // http://bugzilla.opendarwin.org/show_bug.cgi?id=7395#c3
     virtual bool containsFloats() { return m_floatingObjects!=0; }
     virtual bool containsFloat(RenderObject* o);