Let lowestFloatLogicalBottom() take EClear instead of FloatingObject::Type

third_party/WebKit/Source/core/layout/LayoutBlockFlow.h

Index: third_party/WebKit/Source/core/layout/LayoutBlockFlow.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutBlockFlow.h b/third_party/WebKit/Source/core/layout/LayoutBlockFlow.h
index 9fac2d753b52a00a85b96d728ea592998046011a..74f29edad8bde349bf967d9b5e093c3fada4ab17 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBlockFlow.h
+++ b/third_party/WebKit/Source/core/layout/LayoutBlockFlow.h
@@ -379,6 +379,8 @@ class CORE_EXPORT LayoutBlockFlow : public LayoutBlock {
   LayoutUnit lowestFloatLogicalBottom(

[szager1, 2016/11/16 22:35:35]

You should be able to eliminate this method by providing a default for the new
version:

LayoutUnit lowestFloatLogicalBottom(EClear = ClearBoth) const;

[mstensho (USE GERRIT), 2016/11/16 22:45:37]

Indeed! Looks like I didn't bother to check if there were still callers that
passed FloatingObject::Type. Thanks!

Done.

       FloatingObject::Type = FloatingObject::FloatLeftRight) const;
 
+  LayoutUnit lowestFloatLogicalBottom(EClear) const;
+
   bool hasOverhangingFloats() const {
     return parent() && containsFloats() &&
            lowestFloatLogicalBottom() > logicalHeight();