Establish a list marker's offset before floats have been added to its line

third_party/WebKit/Source/core/layout/LayoutListItem.cpp

 /**
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2006 Andrew Wellington (proton@wiretapped.net)
  *
  * 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
  * version 2 of the License, or (at your option) any later version.
@@ skipping 340 matching lines @@
     RootInlineBox& root = m_marker->inlineBoxWrapper()->root();
     bool hitSelfPaintingLayer = false;
 
     LayoutUnit lineTop = root.lineTop();
     LayoutUnit lineBottom = root.lineBottom();
 
     // TODO(jchaffraix): Propagating the overflow to the line boxes seems
     // pretty wrong (https://crbug.com/554160).
     // FIXME: Need to account for relative positioning in the layout overflow.
     if (style()->isLeftToRightDirection()) {
-      LayoutUnit leftLineOffset = logicalLeftOffsetForLine(
-          blockOffset, logicalLeftOffsetForLine(blockOffset, DoNotIndentText),
-          DoNotIndentText);
-      markerLogicalLeft = leftLineOffset - lineOffset - paddingStart() -
+      markerLogicalLeft = m_marker->lineOffset() - lineOffset - paddingStart() -
                           borderStart() + m_marker->marginStart();
       m_marker->inlineBoxWrapper()->moveInInlineDirection(markerLogicalLeft -
                                                           markerOldLogicalLeft);
       for (InlineFlowBox* box = m_marker->inlineBoxWrapper()->parent(); box;
            box = box->parent()) {
         LayoutRect newLogicalVisualOverflowRect =
             box->logicalVisualOverflowRect(lineTop, lineBottom);
         LayoutRect newLogicalLayoutOverflowRect =
             box->logicalLayoutOverflowRect(lineTop, lineBottom);
         if (markerLogicalLeft < newLogicalVisualOverflowRect.x() &&
@@ skipping 11 matching lines @@
           if (box == root)
             adjustOverflow = true;
         }
         box->overrideOverflowFromLogicalRects(newLogicalLayoutOverflowRect,
                                               newLogicalVisualOverflowRect,
                                               lineTop, lineBottom);
         if (box->boxModelObject().hasSelfPaintingLayer())
           hitSelfPaintingLayer = true;
       }
     } else {
-      LayoutUnit rightLineOffset = logicalRightOffsetForLine(
-          blockOffset, logicalRightOffsetForLine(blockOffset, DoNotIndentText),
-          DoNotIndentText);
-      markerLogicalLeft = rightLineOffset - lineOffset + paddingStart() +
+      markerLogicalLeft = m_marker->lineOffset() - lineOffset + paddingStart() +
                           borderStart() + m_marker->marginEnd();
       m_marker->inlineBoxWrapper()->moveInInlineDirection(markerLogicalLeft -
                                                           markerOldLogicalLeft);
       for (InlineFlowBox* box = m_marker->inlineBoxWrapper()->parent(); box;
            box = box->parent()) {
         LayoutRect newLogicalVisualOverflowRect =
             box->logicalVisualOverflowRect(lineTop, lineBottom);
         LayoutRect newLogicalLayoutOverflowRect =
             box->logicalLayoutOverflowRect(lineTop, lineBottom);
         if (markerLogicalLeft + m_marker->logicalWidth() >
@@ skipping 136 matching lines @@
       // assume that all the following ones have too.
       // This gives us the opportunity to stop here and avoid
       // marking the same nodes again.
       break;
     }
     item->updateValue();
   }
 }
 
 }  // namespace blink