[Webkit-unassigned] [Bug 12394] Cleanup RenderBox

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=12394


mitz at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #12644|review?                     |review+
               Flag|                            |




------- Comment #2 from mitz at webkit.org  2007-01-24 12:53 PDT -------
(From update of attachment 12644)
-    if (isFloating() || (isCompact() && isInline()) || 
+    if (isFloating() || (isCompact() && isInline()) ||
         (isInlineBlockOrInlineTable() && !isHTMLMarquee()))

In other places in this patch you moved the operator after the line break. Same
here:

-            m_height = max(m_height, visHeight - 
-                            (margins + parent()->marginTop() +
parent()->marginBottom() + 
+            m_height = max(m_height, visHeight -
+                            (margins + parent()->marginTop() +
parent()->marginBottom() +

and here:

+        for (; !cb->isRenderView() && !cb->isBody() && !cb->isTableCell() &&
!cb->isPositioned() &&
                 cb->style()->height().isAuto(); cb = cb->containingBlock());

r=me


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.