Don't anti-alias backgrounds of images.

Source/core/paint/ImagePainter.cpp

Index: Source/core/paint/ImagePainter.cpp
diff --git a/Source/core/paint/ImagePainter.cpp b/Source/core/paint/ImagePainter.cpp
index 447ccd52abc0db56d7030db011ac1c2e4d8a1e30..04ebee1f5e6d0eee700066c2648c8239f05b760e 100644
--- a/Source/core/paint/ImagePainter.cpp
+++ b/Source/core/paint/ImagePainter.cpp
@@ -200,4 +200,17 @@ void ImagePainter::paintIntoRect(GraphicsContext* context, const LayoutRect& rec
     InspectorInstrumentation::didPaintImage(&m_renderImage);
 }
 
+void ImagePainter::paintBoxDecorationBackground(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
+{
+    // Don't anti-alias the background of an image. See crbug.com/423834 for the reason why.
+    // However, don't turn off anti-aliasing for subclasses such as video.
+    bool shouldAntialias = !m_renderImage.isRenderImage();

[chrishtr, 2014/11/11 19:34:21]

For the record, I also spent a while this morning trying out a generalization of
this patch, by extending the bleed avoidance logic in BoxDecorationData.
That code contains various special cases that are similar in concept to what
this CL is attempting.

Unfortunately, it's hard to get this type of bleed avoidance cleanly into
BoxDecorationData, without adding a new bitfield member to represent "will
eventually have an obscured background". This is because
a) images load asynchronously, so there is a difference between "obscure now"
and "will end up obscured"
b) images may be partially transparent, so we don't even know that the
background will be obscured after load
c) even if we new after load, animated gifs can change transparency between
frames (or at least that's what the code assumes)

All this makes me more comfortable with this targeted approach.

+
+    bool oldShouldAntialias = paintInfo.context->shouldAntialias();
+
+    paintInfo.context->setShouldAntialias(shouldAntialias);
+    BoxPainter(m_renderImage).paintBoxDecorationBackground(paintInfo, paintOffset);
+    paintInfo.context->setShouldAntialias(oldShouldAntialias);
+}
+
 } // namespace blink