Gah. Fix positioned objects and floating objects so that they

	don't append themselves to the list of runs in normal flow.
	An astoundingly basic bug. :(

        Reviewed by john

        * khtml/rendering/bidi.cpp:
        (appendRunsForObject):
        * khtml/rendering/render_flow.cpp:
        (RenderFlow::addChildToFlow):


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3357 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog-2003-10-25

+2003-01-20  David Hyatt  <hyatt@apple.com>
+
+	Gah. Fix positioned objects and floating objects so that they
+	don't append themselves to the list of runs in normal flow.
+	An astoundingly basic bug. :(
+	
+        Reviewed by john
+
+        * khtml/rendering/bidi.cpp:
+        (appendRunsForObject):
+        * khtml/rendering/render_flow.cpp:
+        (RenderFlow::addChildToFlow):
+
 2003-01-19  David Hyatt  <hyatt@apple.com>
 
 	Fixes for 3150561 (blog articles vanish on stylesheet switch) and

WebCore/ChangeLog-2005-08-23

+2003-01-20  David Hyatt  <hyatt@apple.com>
+
+	Gah. Fix positioned objects and floating objects so that they
+	don't append themselves to the list of runs in normal flow.
+	An astoundingly basic bug. :(
+	
+        Reviewed by john
+
+        * khtml/rendering/bidi.cpp:
+        (appendRunsForObject):
+        * khtml/rendering/render_flow.cpp:
+        (RenderFlow::addChildToFlow):
+
 2003-01-19  David Hyatt  <hyatt@apple.com>
 
 	Fixes for 3150561 (blog articles vanish on stylesheet switch) and

WebCore/khtml/rendering/bidi.cpp

@@ -264,7 +264,7 @@ inline QChar::Direction BidiIterator::direction() const
 
 static void appendRunsForObject(int start, int end, RenderObject* obj)
 {
-    if (start > end)
+    if (start > end || obj->isSpecial())
         return;
         
     BidiIterator* nextMidpoint = (smidpoints && smidpoints->count()) ? smidpoints->at(0) : 0;

WebCore/khtml/rendering/render_flow.cpp

@@ -2088,7 +2088,7 @@ void RenderFlow::addChildToFlow(RenderObject* newChild, RenderObject* beforeChil
             }
         }
     }
-    else if (!m_childrenInline)
+    else if (!m_childrenInline && !newChild->isSpecial())
     {
         // If we're inserting an inline child but all of our children are blocks, then we have to make sure
         // it is put into an anomyous block box. We try to use an existing anonymous box if possible, otherwise