Position absolute is not working properly in RTL mode for vertical scroll contents

Position absolute is not working properly in RTL mode for vertical scroll contents

BUG=129746
Positioned elements should consider verticalScrollWidth while calculating position if shouldPlaceBlockDirectionScrollbarOnLogicalLeft() is true.

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=158423

[pals, 2013-09-24 13:47:45]

Please review.

[ojan, 2013-09-25 00:05:13]

lgtm

[commit-bot: I haz the power, 2013-09-25 00:07:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sanjoy.pal@samsung.com/24365011/1

[commit-bot: I haz the power, 2013-09-25 00:07:18]

Failed to apply patch for LayoutTests/TestExpectations:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file LayoutTests/TestExpectations
  Hunk #1 FAILED at 1349.
  1 out of 1 hunk FAILED -- saving rejects to file
LayoutTests/TestExpectations.rej

Patch:       LayoutTests/TestExpectations
Index: LayoutTests/TestExpectations
diff --git a/LayoutTests/TestExpectations b/LayoutTests/TestExpectations
index
4498363deed10f94eaae558b82a42fe75a2c04cd..565ba0527eff01a5f5a6445698ea1faa1a17632f
100644
--- a/LayoutTests/TestExpectations
+++ b/LayoutTests/TestExpectations
@@ -1349,3 +1349,4 @@ crbug.com/289833 [ Android ]
webexposed/custom-elements.html [ Timeout Pass ]
 # These started asserting with blink r157905. The asserts are symtpoms of
legitimate bugs.
 crbug.com/294237 [ Debug ] svg/custom/bug86392.html [ Crash ]
 crbug.com/294238 [ Debug ] svg/custom/unicode-in-tspan-multi-svg-crash.html [
Crash ]
+crbug.com/129746 [ Mac ] fast/overflow/unreachable-overflow-rtl-bug.html [
NeedsRebaseline ]

[commit-bot: I haz the power, 2013-09-25 07:14:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sanjoy.pal@samsung.com/24365011/7001

[commit-bot: I haz the power, 2013-09-25 09:11:56]

Retried try job too often on linux_layout for step(s) webkit_lint
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_layo...

[commit-bot: I haz the power, 2013-09-26 11:31:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sanjoy.pal@samsung.com/24365011/23001

[commit-bot: I haz the power, 2013-09-26 13:41:30]

Retried try job too often on win_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_blink_...

[commit-bot: I haz the power, 2013-09-27 06:00:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sanjoy.pal@samsung.com/24365011/38001

[commit-bot: I haz the power, 2013-09-27 07:14:01]

Change committed as 158423