BR styled clear is ignored while partially relayouting.

Source/core/layout/LayoutBlockFlowLine.cpp

Index: Source/core/layout/LayoutBlockFlowLine.cpp
diff --git a/Source/core/layout/LayoutBlockFlowLine.cpp b/Source/core/layout/LayoutBlockFlowLine.cpp
index f50503df8f4ac7314d450abc495ec79108cc2260..afaf3a8a017148e3f4034ed1b872791deb5fd106 100644
--- a/Source/core/layout/LayoutBlockFlowLine.cpp
+++ b/Source/core/layout/LayoutBlockFlowLine.cpp
@@ -1563,6 +1563,9 @@ void LayoutBlockFlow::layoutInlineChildren(bool relayoutChildren, LayoutUnit& pa
             if (!layoutState.hasInlineChild() && o->isInline())
                 layoutState.setHasInlineChild(true);
 
+            if (o->isBR() && o->style()->clear() != CNONE)

[leviw_travelin_and_unemployed, 2015/05/14 02:22:19]

It's sad that we're doing this whenever there's any br with a clear, not only if
they're in the end of the line.

That said, I think this patch is unintrusive enough as to be okay.

[changseok, 2015/05/14 16:18:46]

Yeah.. Do you have any better idea to reduce iteration? I always welcome a neat
idea. =)

+                layoutState.setContainsBRWithClear(true);
+
             if (o->isReplaced() || o->isFloating() || o->isOutOfFlowPositioned()) {
                 LayoutBox* box = toLayoutBox(o);
 
@@ -1856,7 +1859,7 @@ bool LayoutBlockFlow::checkPaginationAndFloatsAtEndLine(LineLayoutState& layoutS
 bool LayoutBlockFlow::matchedEndLine(LineLayoutState& layoutState, const InlineBidiResolver& resolver, const InlineIterator& endLineStart, const BidiStatus& endLineStatus)
 {
     if (resolver.position() == endLineStart) {
-        if (resolver.status() != endLineStatus)
+        if (resolver.status() != endLineStatus || (containsFloats() && layoutState.containsBRWithClear()))

[leviw_travelin_and_unemployed, 2015/05/14 02:22:19]

I think I'd prefer the following:

if (resolver.status() != endLineStatus)
     return false;

// Comment about why we're doing this
if (layoutState.containsBRWithClear() && containsFloats())
    return false;

You'll notice I reversed the order of these because I suspect containing br's
with clear is less common than floats.

[changseok, 2015/05/14 16:18:46]

Make sense.

             return false;
         return checkPaginationAndFloatsAtEndLine(layoutState);
     }