[S.P.] Include selection rect in list marker cull rect when needed

Source/core/paint/ListMarkerPainter.cpp

Index: Source/core/paint/ListMarkerPainter.cpp
diff --git a/Source/core/paint/ListMarkerPainter.cpp b/Source/core/paint/ListMarkerPainter.cpp
index 7caf6846ae117194f5e9f4208554c3ce2c8ffcc6..36790510049be0d377e038605c882375dd22e990 100644
--- a/Source/core/paint/ListMarkerPainter.cpp
+++ b/Source/core/paint/ListMarkerPainter.cpp
@@ -27,6 +27,8 @@ void ListMarkerPainter::paint(const PaintInfo& paintInfo, const LayoutPoint& pai
 
     LayoutPoint boxOrigin(paintOffset + m_layoutListMarker.location());
     LayoutRect overflowRect(m_layoutListMarker.visualOverflowRect());
+    if (m_layoutListMarker.selectionState() != LayoutObject::SelectionNone)
+        overflowRect.unite(m_layoutListMarker.localSelectionRect());

[chrishtr, 2015/06/04 18:13:22]

I wonder if the other classes that have a special localSelectionRect will need
this code. In particular, LayoutReplaced, LayoutText, InlineTextBox and
SVGInlineTextBox.

[fs, 2015/06/05 08:56:19]

Most likely - I added one for InlineTextBox last week or so ago; LayoutReplaced
seems covered (ReplacedPainter, PartPainter); LayoutText doesn't have any AFAICS
(and I wouldn't expect it to), which leaves SVGInlineTextBox (where this method
can be pretty expensive =/).

     overflowRect.moveBy(boxOrigin);
 
     IntRect pixelSnappedOverflowRect = pixelSnappedIntRect(overflowRect);