Split list marker updating into two steps

Source/core/rendering/RenderListItem.h

Index: Source/core/rendering/RenderListItem.h
diff --git a/Source/core/rendering/RenderListItem.h b/Source/core/rendering/RenderListItem.h
index 3d6edc57c94c891a20200eefa9d0a7dd0207661c..732e218e3d019e0d01cb35f8c60cdd2274075ad8 100644
--- a/Source/core/rendering/RenderListItem.h
+++ b/Source/core/rendering/RenderListItem.h
@@ -42,13 +42,12 @@ public:
     void setExplicitValue(int value);
     void clearExplicitValue();
 
-    void setNotInList(bool notInList) { m_notInList = notInList; }
+    void setNotInList(bool);
     bool notInList() const { return m_notInList; }
 
     const String& markerText() const;
 
     void updateListMarkerNumbers();
-    void updateMarkerLocation();
 
     static void updateItemValuesForOrderedList(const HTMLOListElement*);
     static unsigned itemCountForOrderedList(const HTMLOListElement*);
@@ -69,6 +68,9 @@ private:
 
     virtual void layout() OVERRIDE;
 
+    void updateMarkerAndInvalidateWidth();
+    bool updateMarker();

[skobes, 2014/07/02 05:59:42]

Suggest a comment explaining the meaning of the return value.

[leviw_travelin_and_unemployed, 2014/07/02 18:07:14]

I may call this updateMarkerLocation still, since it primarily moves the marker
around.

[pdr., 2014/07/02 18:39:30]

Done.

[pdr., 2014/07/02 18:39:30]

Done.

+
     void positionListMarker();
 
     virtual void styleDidChange(StyleDifference, const RenderStyle* oldStyle) OVERRIDE;