From ec084dc47af8aa17400ef179bf039f9fb72f4c0e Mon Sep 17 00:00:00 2001 From: Ojan Vafai Date: Tue, 11 Nov 2014 16:58:33 -0800 Subject: [PATCH] Make paddingBoxRect less dumb. It was calling contentWidth, which did clientWidth - padding, and then adding the padding back in. R=esprehn@chromium.org Review URL: https://codereview.chromium.org/715053004 --- engine/core/rendering/RenderBox.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/engine/core/rendering/RenderBox.h b/engine/core/rendering/RenderBox.h index 4446622e158..bfed6c7c439 100644 --- a/engine/core/rendering/RenderBox.h +++ b/engine/core/rendering/RenderBox.h @@ -162,7 +162,7 @@ public: void setFrameRect(const LayoutRect& rect) { m_frameRect = rect; } LayoutRect borderBoxRect() const { return LayoutRect(LayoutPoint(), size()); } - LayoutRect paddingBoxRect() const { return LayoutRect(borderLeft(), borderTop(), contentWidth() + paddingLeft() + paddingRight(), contentHeight() + paddingTop() + paddingBottom()); } + LayoutRect paddingBoxRect() const { return LayoutRect(borderLeft(), borderTop(), clientWidth(), clientHeight()); } IntRect pixelSnappedBorderBoxRect() const { return IntRect(IntPoint(), m_frameRect.pixelSnappedSize()); } virtual IntRect borderBoundingBox() const override final { return pixelSnappedBorderBoxRect(); }