Position a float before laying it out.

third_party/WebKit/Source/core/layout/line/BreakingContextInlineHeaders.h

 /*
  * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2003, 2004, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc.
  *               All right reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated.
  *
  * 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 470 matching lines @@
     positionedObjects.append(box);
   m_width.addUncommittedWidth(
       inlineLogicalWidthFromAncestorsIfNeeded(box).toFloat());
   // Reset prior line break context characters.
   m_layoutTextInfo.m_lineBreakIterator.resetPriorContext();
 }
 
 inline void BreakingContext::handleFloat() {
   LineLayoutBox floatBox(m_current.getLineLayoutItem());
   FloatingObject* floatingObject = m_block.insertFloatingObject(floatBox);
-  // Check if it fits in the current line; if it does, position it now,
-  // otherwise, position it after moving to next line (in newLine() func).
-  // FIXME: Bug 110372: Properly position multiple stacked floats with
-  // non-rectangular shape outside.
-  if (m_floatsFitOnLine &&
-      m_width.fitsOnLine(
-          m_block.logicalWidthForFloat(*floatingObject).toFloat(),
-          ExcludeWhitespace)) {
-    m_block.placeNewFloats(m_block.logicalHeight(), &m_width);
-    if (m_lineBreak.getLineLayoutItem() == m_current.getLineLayoutItem()) {
-      ASSERT(!m_lineBreak.offset());
-      m_lineBreak.increment();
+
+  if (m_floatsFitOnLine) {
+    // We need to calculate the logical width of the float before we can tell
+    // whether it's going to fit on the line.
+    m_block.positionAndLayoutFloat(*floatingObject, m_block.logicalHeight());
+
+    // Check if it fits in the current line; if it does, place it now,
+    // otherwise, place it after moving to next line (in newLine() func).
+    // FIXME: Bug 110372: Properly position multiple stacked floats with
+    // non-rectangular shape outside.
+    if (m_width.fitsOnLine(
+            m_block.logicalWidthForFloat(*floatingObject).toFloat(),
+            ExcludeWhitespace)) {
+      m_block.placeNewFloats(m_block.logicalHeight(), &m_width);
+      if (m_lineBreak.getLineLayoutItem() == m_current.getLineLayoutItem()) {
+        DCHECK(!m_lineBreak.offset());
+        m_lineBreak.increment();
+      }
+    } else {
+      m_floatsFitOnLine = false;
     }
-  } else {
-    m_floatsFitOnLine = false;
   }
   // Update prior line break context characters, using U+FFFD (OBJECT
   // REPLACEMENT CHARACTER) for floating element.
   m_layoutTextInfo.m_lineBreakIterator.updatePriorContext(replacementCharacter);
 }
 
 // This is currently just used for list markers and inline flows that have line
 // boxes. Neither should have an effect on whitespace at the start of the line.
 inline bool shouldSkipWhitespaceAfterStartObject(
     LineLayoutBlockFlow block,
@@ skipping 975 matching lines @@
 
   if (style.getTextIndentType() == TextIndentHanging)
     indentText = indentText == IndentText ? DoNotIndentText : IndentText;
 
   return indentText;
 }
 
 }  // namespace blink
 
 #endif  // BreakingContextInlineHeaders_h