Correction to layoutOverflowRect in case of direction:rtl

Source/core/rendering/RenderBox.h

Index: Source/core/rendering/RenderBox.h
diff --git a/Source/core/rendering/RenderBox.h b/Source/core/rendering/RenderBox.h
index 88d627d6e3ebbe547f0ec58df32dc3b872068aca..be6c0aff9454427f006c5afd85e1ce52c5bf86fc 100644
--- a/Source/core/rendering/RenderBox.h
+++ b/Source/core/rendering/RenderBox.h
@@ -221,7 +221,7 @@ public:
 
     // More IE extensions.  clientWidth and clientHeight represent the interior of an object
     // excluding border and scrollbar.  clientLeft/Top are just the borderLeftWidth and borderTopWidth.
-    LayoutUnit clientLeft() const { return borderLeft(); }
+    LayoutUnit clientLeft() const { return borderLeft() + (style()->shouldPlaceBlockDirectionScrollbarOnLogicalLeft() ? verticalScrollbarWidth() : 0); }

[ojan, 2013/10/01 19:24:52]

This changes the web exposed behavior of clientLeft. Does this not affect
existing test cases? What do other browsers do in this case?

[pals, 2013/10/03 10:54:06]

Removed the change from this patch. I shall create separate CL to fix
crbug.com/128500. Yes, this changes the web exposed behavior of clientLeft. It
seems we dont have a test case for clientLeft. Please see comment #4 in the bug
for more details on other browsers behavior.

     LayoutUnit clientTop() const { return borderTop(); }
     LayoutUnit clientWidth() const;
     LayoutUnit clientHeight() const;