List marker pseudo elements.

Source/core/dom/MarkerPseudoElement.cpp

+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "config.h"
+#include "core/dom/MarkerPseudoElement.h"
+
+#include "core/layout/LayoutBlockFlow.h"
+#include "core/layout/LayoutListItem.h"
+#include "core/layout/LayoutListMarker.h"
+
+namespace blink {
+
+MarkerPseudoElement::MarkerPseudoElement(Element* parent)
+    : PseudoElement(parent, MARKER)
+{
+}
+
+MarkerPseudoElement::~MarkerPseudoElement()
+{
+}
+
+LayoutObject* MarkerPseudoElement::getParentOfFirstLineBox(LayoutBlockFlow* curr, const LayoutObject* marker)

[esprehn, 2015/04/20 16:04:08]

Remove get prefix. 

[dsinclair, 2015/04/21 20:23:10]

Done.

+{
+    LayoutObject* firstChild = curr->firstChild();
+    if (!firstChild)
+        return nullptr;
+
+    bool inQuirksMode = curr->document().inQuirksMode();
+    for (LayoutObject* currChild = firstChild; currChild; currChild = currChild->nextSibling()) {
+        if (currChild == marker)
+            continue;
+
+        if (currChild->isInline() && (!currChild->isLayoutInline() || curr->generatesLineBoxesForInlineChild(currChild)))
+            return curr;
+
+        if (currChild->isFloating() || currChild->isOutOfFlowPositioned())
+            continue;
+
+        if (!currChild->isLayoutBlockFlow() || (currChild->isBox() && toLayoutBox(currChild)->isWritingModeRoot()))
+            break;
+
+        if (curr->isListItem() && inQuirksMode && currChild->node()
+            && (isHTMLUListElement(*currChild->node()) || isHTMLOListElement(*currChild->node())))
+            break;
+
+        if (LayoutObject* lineBox = getParentOfFirstLineBox(toLayoutBlockFlow(currChild), marker))
+            return lineBox;
+    }
+
+    return nullptr;
+}
+
+PassRefPtr<ComputedStyle> MarkerPseudoElement::customStyleForLayoutObject()
+{
+    if (!layoutObject())
+        return nullptr;
+
+    LayoutObject* parent = layoutObject()->parent();
+    if (!parent)
+        return nullptr;
+
+    return styleForListMarkerLayoutObject(*parent, layoutObject()->style());

[esprehn, 2015/04/20 16:04:08]

This isn't right,  you're rebuilding what is Inheritance style changes.  This is
also unstable, when you first attach this returns null and then later starts
returning a value. 

[dsinclair, 2015/04/21 20:23:10]

I'm not sure I understand, what is the correct way to do this?

The code in styleForListMarkerLayoutObject is moved from
LayoutListMarker::listItemStyleDidChange() and does the same as it did
previously. This method is just calling down to that in order to retrieve the
style.

+}
+
+PassRefPtr<ComputedStyle> MarkerPseudoElement::styleForListMarkerLayoutObject(LayoutObject& parent, const ComputedStyle* style)
+{
+    RefPtr<ComputedStyle> newStyle = ComputedStyle::create();
+
+    // The marker always inherits from the list item, regardless of where it might end
+    // up (e.g., in some deeply nested line box). See CSS3 spec.
+    newStyle->inheritFrom(*parent.style());
+    if (style) {
+        // Reuse the current margins. Otherwise resetting the margins to initial values
+        // would trigger unnecessary layout.
+        newStyle->setMarginStart(style->marginStart());
+        newStyle->setMarginEnd(style->marginRight());
+    }
+    return newStyle;
+}
+
+LayoutObject* MarkerPseudoElement::firstNonMarkerChild(LayoutObject* parent)
+{
+    LayoutObject* result = parent->slowFirstChild();
+    while (result && result->isListMarker())
+        result = result->nextSibling();
+    return result;
+}
+
+LayoutObject* MarkerPseudoElement::createLayoutObject(const ComputedStyle& style)
+{
+    ASSERT(parentElement()->layoutObject());
+    LayoutListMarker* marker = new LayoutListMarker(this, toLayoutListItem(parentElement()->layoutObject()));
+    marker->setStyle(ComputedStyle::clone(style));
+    return marker;
+}
+
+void MarkerPseudoElement::attachListMarkerLayoutObject()
+{
+    LayoutListItem* parentLayoutObject = toLayoutListItem(parentElement()->layoutObject());
+    ASSERT(parentLayoutObject);
+    ASSERT(layoutObject());
+
+    LayoutListMarker* marker = toLayoutListMarker(layoutObject());
+    LayoutObject* markerParent = marker->parent();
+
+    LayoutObject* insertionLayoutObject = MarkerPseudoElement::getParentOfFirstLineBox(parentLayoutObject, marker);
+
+    // We didn't find any line boxes so make the insertion point the LI layoutObject.
+    if (!insertionLayoutObject)
+        insertionLayoutObject = parentLayoutObject;
+    ASSERT(insertionLayoutObject);
+
+    // Check if we're already inserted into the right parent as the first child.
+    if (markerParent && marker == markerParent->slowFirstChild()
+        && (markerParent == insertionLayoutObject || markerParent == insertionLayoutObject->slowFirstChild()))
+        return;
+
+    marker->remove();
+    insertionLayoutObject->addChild(marker, MarkerPseudoElement::firstNonMarkerChild(insertionLayoutObject));
+    marker->updateMarginsAndContent();
+
+    // If markerParent is an anonymous block with no children, destroy it.
+    if (markerParent && markerParent->isAnonymousBlock() && !toLayoutBlock(markerParent)->firstChild() && !toLayoutBlock(markerParent)->continuation())
+        markerParent->destroy();
+
+    if (marker->isInside())
+        parentLayoutObject->containingBlock()->updateLogicalWidth();
+}
+
+void MarkerPseudoElement::attach(const AttachContext& context)
+{
+    PseudoElement::attach(context);
+    ASSERT(layoutObject());
+
+    LayoutListMarker* marker = toLayoutListMarker(layoutObject());
+
+    RefPtr<ComputedStyle> newStyle = MarkerPseudoElement::styleForListMarkerLayoutObject(*(marker->mutableListItem()), marker->style());
+    marker->setStyle(newStyle.release());
+
+    attachListMarkerLayoutObject();
+}
+
+void MarkerPseudoElement::didRecalcStyle(StyleRecalcChange)
+{
+    attachListMarkerLayoutObject();
+}
+
+} // namespace blink