Don't paint below image's baseline unless it is selected.

Source/core/rendering/RenderReplaced.h

Index: Source/core/rendering/RenderReplaced.h
diff --git a/Source/core/rendering/RenderReplaced.h b/Source/core/rendering/RenderReplaced.h
index 030e2c870d2ae0a93b9cf4636461233418966787..dcd863edb192837a07b9abcb2937304abceeacbe 100644
--- a/Source/core/rendering/RenderReplaced.h
+++ b/Source/core/rendering/RenderReplaced.h
@@ -69,6 +69,8 @@ protected:
     LayoutRect localSelectionRect(bool checkWhetherSelected = true) const; // This is in local coordinates, but it's a physical rect (so the top left corner is physical top left).
     virtual RenderBox* embeddedContentBox() const { return 0; }
 
+    virtual LayoutRect overflowRectForRepaint() const OVERRIDE FINAL { return isSelected() ? localSelectionRect() : visualOverflowRect(); }

[Julien - ping for review, 2014/05/02 00:23:35]

I wonder if it would make sense for the visualOverflowRect to just include the
selection rect. The downside of that would be that you would need to recompute
it whenever selection changes but I think it's an OK trade off.

Also is this change really required as it seems orthogonal to the fix which
would be the computeRepaintRect when changing the selection state?

[rhogan, 2014/05/02 15:36:27]

That's what we do currently for replaced objects - paint the union of the
selection rect and the overflow rect. That's unnecessary invalidation unless the
image is selected. If it isn't selected we just want to invalidate the
visualOverflowRect() - which is the motivation of this change.
The call to computeRepaintRects() when changing selection state is required in
order to keep the cached repaint rect up to date for images with a layer. Images
without a layer get it when they need it from overflowRectForRepaint().

+
 private:
     virtual const char* renderName() const OVERRIDE { return "RenderReplaced"; }
 
@@ -77,8 +79,6 @@ private:
     virtual void computePreferredLogicalWidths() OVERRIDE FINAL;
     virtual void paintReplaced(PaintInfo&, const LayoutPoint&) { }
 
-    virtual LayoutRect clippedOverflowRectForRepaint(const RenderLayerModelObject* repaintContainer) const OVERRIDE;
-
     virtual PositionWithAffinity positionForPoint(const LayoutPoint&) OVERRIDE FINAL;
 
     virtual bool canBeSelectionLeaf() const OVERRIDE { return true; }