commit 69340905255e2d0eed43dc83df223f781db1f0c2
author hyatt@apple.com <hyatt@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc> Wed Jan 16 21:24:21 2008 +0000
committer hyatt@apple.com <hyatt@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc> Wed Jan 16 21:24:21 2008 +0000
tree 041c26aaa781cc9ffe72528e5a156173e1099410
parent cd9b7bea3bbc1e681e62d3644d0c1f2c8f47ed19

I don't know what happened here. It did not commit with my checkin.

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

WebCore/rendering/bidi.cpp

diff --git a/WebCore/rendering/bidi.cpp b/WebCore/rendering/bidi.cpp
index 6a05f95..56e63d8 100644
--- a/WebCore/rendering/bidi.cpp
+++ b/WebCore/rendering/bidi.cpp
@@ -1395,6 +1395,11 @@
 
     bool autoWrapWasEverTrueOnLine = false;
     
+    // Firefox and Opera will allow a table cell to grow to fit an image inside it under
+    // very specific cirucumstances (in order to match common WinIE renderings). 
+    // Not supporting the quirk has caused us to mis-render some real sites. (See Bugzilla 10517.) 
+    bool allowImagesToBreak = !style()->htmlHacks() || !isTableCell() || !style()->width().isIntrinsicOrAuto();
+
     EWhiteSpace currWS = style()->whiteSpace();
     EWhiteSpace lastWS = currWS;
     while (o) {
@@ -1520,7 +1525,7 @@
                     o->marginRight() + o->borderRight() + o->paddingRight();
         } else if (o->isReplaced()) {
             // Break on replaced elements if either has normal white-space.
-            if (autoWrap || RenderStyle::autoWrap(lastWS)) {
+            if ((autoWrap || RenderStyle::autoWrap(lastWS)) && (!o->isImage() || allowImagesToBreak)) {
                 w += tmpW;
                 tmpW = 0;
                 lBreak.obj = o;
@@ -1874,7 +1879,7 @@
             previous = o;
         o = next;
 
-        if (!last->isFloatingOrPositioned() && last->isReplaced() && autoWrap && 
+        if (!last->isFloatingOrPositioned() && last->isReplaced() && autoWrap && (!last->isImage() || allowImagesToBreak) &&
             (!last->isListMarker() || static_cast<RenderListMarker*>(last)->isInside())) {
             w += tmpW;
             tmpW = 0;