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

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

 // 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.
 
 #ifndef ImagePainter_h
 #define ImagePainter_h
 
 #include "wtf/Allocator.h"
 
 namespace blink {
 
 class GraphicsContext;
 struct PaintInfo;
 class LayoutPoint;
 class LayoutRect;
 class LayoutImage;
 
 class ImagePainter {
     STACK_ALLOCATED();
 public:
     ImagePainter(const LayoutImage& layoutImage) : m_layoutImage(layoutImage) { }
 
     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);
 
     const LayoutImage& m_layoutImage;
 };
 
 } // namespace blink
 
 #endif // ImagePainter_h