Always include floating children into visual overflow

Source/core/layout/LayoutBlock.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 913 matching lines @@
 }
 
 void LayoutBlock::addOverflowFromChildren()
 {
     if (childrenInline())
         toLayoutBlockFlow(this)->addOverflowFromInlineChildren();
     else
         addOverflowFromBlockChildren();
 }
 
-void LayoutBlock::computeOverflow(LayoutUnit oldClientAfterEdge, bool)
+void LayoutBlock::computeOverflow(LayoutUnit oldClientAfterEdge)
 {
     m_overflow.clear();
 
     // Add overflow from children.
     addOverflowFromChildren();
 
     // Add in the overflow from positioned objects.
     addOverflowFromPositionedObjects();
 
     if (hasOverflowClip()) {
@@ skipping 144 matching lines @@
             layoutPositionedObjects(false, needsPositionedMovementLayout() ? ForcedLayoutAfterContainingBlockMoved : (!posChildNeedsLayout() && canContainFixedPosObjects ? LayoutOnlyFixedPositionedObjects : DefaultLayout));
 
         // Recompute our overflow information.
         // FIXME: We could do better here by computing a temporary overflow object from layoutPositionedObjects and only
         // updating our overflow if we either used to have overflow or if the new temporary object has overflow.
         // For now just always recompute overflow. This is no worse performance-wise than the old code that called rightmostPosition and
         // lowestPosition on every relayout so it's not a regression.
         // computeOverflow expects the bottom edge before we clamp our height. Since this information isn't available during
         // simplifiedLayout, we cache the value in m_overflow.
         LayoutUnit oldClientAfterEdge = hasOverflowModel() ? m_overflow->layoutClientAfterEdge() : clientLogicalBottom();
-        computeOverflow(oldClientAfterEdge, true);
+        computeOverflow(oldClientAfterEdge);
     }
 
     updateLayerTransformAfterLayout();
 
     updateScrollInfoAfterLayout();
 
     clearNeedsLayout();
 
     if (LayoutAnalyzer* analyzer = frameView()->layoutAnalyzer())
         analyzer->increment(LayoutAnalyzer::LayoutObjectsThatNeedSimplifiedLayout);
@@ skipping 1701 matching lines @@
     if (!selfNeedsOverflowRecalcAfterStyleChange() && !childrenOverflowChanged)
         return false;
 
     setSelfNeedsOverflowRecalcAfterStyleChange(false);
     // If the current block needs layout, overflow will be recalculated during
     // layout time anyway. We can safely exit here.
     if (needsLayout())
         return false;
 
     LayoutUnit oldClientAfterEdge = hasOverflowModel() ? m_overflow->layoutClientAfterEdge() : clientLogicalBottom();
-    computeOverflow(oldClientAfterEdge, true);
+    computeOverflow(oldClientAfterEdge);
 
     if (hasOverflowClip())
         layer()->scrollableArea()->updateAfterOverflowRecalc();
 
     return !hasOverflowClip();
 }
 
 // Called when a positioned object moves but doesn't necessarily change size.  A simplified layout is attempted
 // that just updates the object's position. If the size does change, the object remains dirty.
 bool LayoutBlock::tryLayoutDoingPositionedMovementOnly()
@@ skipping 63 matching lines @@
 void LayoutBlock::showLineTreeAndMark(const InlineBox* markedBox1, const char* markedLabel1, const InlineBox* markedBox2, const char* markedLabel2, const LayoutObject* obj) const
 {
     showLayoutObject();
     for (const RootInlineBox* root = firstRootBox(); root; root = root->nextRootBox())
         root->showLineTreeAndMark(markedBox1, markedLabel1, markedBox2, markedLabel2, obj, 1);
 }
 
 #endif
 
 } // namespace blink