Adjust src and dest rects when drawing images instead of using a clip

third_party/WebKit/Source/core/paint/ImagePainter.h

Index: third_party/WebKit/Source/core/paint/ImagePainter.h
diff --git a/third_party/WebKit/Source/core/paint/ImagePainter.h b/third_party/WebKit/Source/core/paint/ImagePainter.h
index fce4d6185bb065dcd7e4ef16367f7c2f03af664b..20c4ea152ddec94172ed03aadf11c2efa3efca74 100644
--- a/third_party/WebKit/Source/core/paint/ImagePainter.h
+++ b/third_party/WebKit/Source/core/paint/ImagePainter.h
@@ -22,7 +22,9 @@ public:
 
     void paint(const PaintInfo&, const LayoutPoint& paintOffset);
     void paintReplaced(const PaintInfo&, const LayoutPoint& paintOffset);
-    void paintIntoRect(GraphicsContext&, const LayoutRect&);
+
+    // Paint the full image into the dest, after clipping by the content rect.

[chrishtr, 2016/07/11 18:11:24]

"into |destRect|, after clipping by |contentRect|"

Also, it's unclear from context here what you mean by "paint into destRect", and
you should specify the coordinate space of these rects.

[pdr., 2016/07/11 19:58:17]

Done.
We talked about this offline but when I tried to do the refactoring to also pass
PaintOffset, I found it duplicated too much code. I've updated this comment to
say:
// Both |destRect| and |contentRect| should already be moved by the paint
offset.
This doesn't exactly answer the question about what coordinate space contentRect
is in though.

+    void paintIntoRect(GraphicsContext&, const LayoutRect& destRect, const LayoutRect& contentRect);
 
 private:
     void paintAreaElementFocusRing(const PaintInfo&, const LayoutPoint& paintOffset);