Fix layoutOverflowRect in case of explicit 'overflow: scroll', but no overflow content

Source/core/rendering/RenderBox.cpp

Index: Source/core/rendering/RenderBox.cpp
diff --git a/Source/core/rendering/RenderBox.cpp b/Source/core/rendering/RenderBox.cpp
index c1e5be5dd91567fc3f2fe9a50c3f3e0474d08727..28b135dd0fe110afcbf93c914c96ade9e8077aed 100644
--- a/Source/core/rendering/RenderBox.cpp
+++ b/Source/core/rendering/RenderBox.cpp
@@ -4269,12 +4269,8 @@ void RenderBox::addContentsVisualOverflow(const LayoutRect& rect)
         return;
     }
 
-    if (!m_overflow) {
-        LayoutRect clientBox = clientBoxRect();
-        if (style()->shouldPlaceBlockDirectionScrollbarOnLogicalLeft())
-            clientBox.move(-verticalScrollbarWidth(), 0);
-        m_overflow = adoptPtr(new RenderOverflow(clientBox, borderBoxRect()));
-    }
+    if (!m_overflow)
+        m_overflow = adoptPtr(new RenderOverflow(clientBoxRect(), borderBoxRect()));

[Julien - ping for review, 2014/05/19 14:36:19]

This should be noOverflowRect() to match the rest of the code.

[pals, 2014/05/20 09:20:19]

Done.

     m_overflow->addContentsVisualOverflow(rect);
 }