Adjust color for printing list-item markers as we do for text.

third_party/WebKit/Source/core/paint/ListMarkerPainter.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 "core/paint/ListMarkerPainter.h"
 
 #include "core/layout/LayoutListItem.h"
 #include "core/layout/LayoutListMarker.h"
 #include "core/layout/ListMarkerText.h"
 #include "core/layout/api/SelectionState.h"
+#include "core/paint/BoxPainter.h"
 #include "core/paint/LayoutObjectDrawingRecorder.h"
 #include "core/paint/PaintInfo.h"
+#include "core/paint/TextPainter.h"
 #include "platform/geometry/LayoutPoint.h"
 #include "platform/graphics/GraphicsContextStateSaver.h"
 
 namespace blink {
 
 static inline void paintSymbol(GraphicsContext& context, const Color& color,
     const IntRect& marker, EListStyleType listStyle)
 {
     context.setStrokeColor(color);
     context.setStrokeStyle(SolidStroke);
@@ skipping 50 matching lines @@
             selRect.moveBy(boxOrigin);
             context.fillRect(pixelSnappedIntRect(selRect), m_layoutListMarker.listItem()->selectionBackgroundColor());
         }
         return;
     }
 
     LayoutListMarker::ListStyleCategory styleCategory = m_layoutListMarker.getListStyleCategory();
     if (styleCategory == LayoutListMarker::ListStyleCategory::None)
         return;
 
-    const Color color(m_layoutListMarker.resolveColor(CSSPropertyColor));
+    Color color(m_layoutListMarker.resolveColor(CSSPropertyColor));
+
+    if (BoxPainter::shouldForceWhiteBackgroundForPrintEconomy(m_layoutListMarker.styleRef(), m_layoutListMarker.listItem()->document()))

[fs, 2016/06/01 08:50:10]

(I guess this could just be m_layoutListMarker.document(), it should handle the
anonymous nature of the marker. I guess this might be more direct though.)

[rune, 2016/06/01 09:43:31]

They are currently always anonymous, so LayoutObject::document() would then
always check m_node and call parent().

Are you fine with keeping this as it is?

[fs, 2016/06/01 10:22:59]

Yes.

+        color = TextPainter::textColorForWhiteBackground(color);
+
     // Apply the color to the list marker text.
     context.setFillColor(color);
 
     const EListStyleType listStyle = m_layoutListMarker.style()->listStyleType();
     if (styleCategory == LayoutListMarker::ListStyleCategory::Symbol) {
         paintSymbol(context, color, marker, listStyle);
         return;
     }
 
     if (m_layoutListMarker.text().isEmpty())
@@ skipping 39 matching lines @@
     if (m_layoutListMarker.style()->isLeftToRightDirection()) {
         context.drawText(font, textRunPaintInfo, textOrigin);
         context.drawText(font, suffixRunInfo, textOrigin + IntSize(font.width(textRun), 0));
     } else {
         context.drawText(font, suffixRunInfo, textOrigin);
         context.drawText(font, textRunPaintInfo, textOrigin + IntSize(font.width(suffixRun), 0));
     }
 }
 
 } // namespace blink