Chromium Code Reviews| Index: Source/core/rendering/RenderImage.h |
| diff --git a/Source/core/rendering/RenderImage.h b/Source/core/rendering/RenderImage.h |
| index 33adcf0adde82c3bdc3609b9b85c75bc784c940e..91f377ae00fca3cf0c86416f2fb40bffb34d5a20 100644 |
| --- a/Source/core/rendering/RenderImage.h |
| +++ b/Source/core/rendering/RenderImage.h |
| @@ -35,6 +35,10 @@ class HTMLMapElement; |
| class RenderImage : public RenderReplaced { |
| public: |
| + // If we'll be displaying either alt text or an image, add some padding. |
|
Stephen Chennney
2014/09/24 15:08:18
I would rather "The padding to use when ..."
chrishtr
2014/09/24 16:30:26
Done.
|
| + static const unsigned short paddingWidth = 4; |
| + static const unsigned short paddingHeight = 4; |
|
Stephen Chennney
2014/09/24 15:08:18
Why not put these in the painter class?
chrishtr
2014/09/24 16:30:25
They are also referred to in this class, for layou
|
| + |
| RenderImage(Element*); |
| virtual ~RenderImage(); |
| virtual void destroy() OVERRIDE; |
| @@ -78,7 +82,6 @@ protected: |
| virtual void imageChanged(WrappedImagePtr, const IntRect* = 0) OVERRIDE; |
| - void paintIntoRect(GraphicsContext*, const LayoutRect&); |
| virtual void paint(PaintInfo&, const LayoutPoint&) OVERRIDE FINAL; |
| virtual void layout() OVERRIDE; |
| virtual bool updateImageLoadingPriorities() OVERRIDE FINAL; |
| @@ -107,8 +110,6 @@ private: |
| // Update the size of the image to be rendered. Object-fit may cause this to be different from the CSS box's content rect. |
| void updateInnerContentRect(); |
| - void paintAreaElementFocusRing(PaintInfo&); |
| - |
| // Text to display as long as the image isn't available. |
| String m_altText; |
| OwnPtr<RenderImageResource> m_imageResource; |