Index: Source/core/frame/FrameView.cpp |
diff --git a/Source/core/frame/FrameView.cpp b/Source/core/frame/FrameView.cpp |
index 95949a17ce21191ed4eea0afd6e8b3e0e780bee1..9f1aaef118b42c813b72f6ef4ce728f9b7bc803e 100644 |
--- a/Source/core/frame/FrameView.cpp |
+++ b/Source/core/frame/FrameView.cpp |
@@ -1329,6 +1329,22 @@ LayoutRect FrameView::viewportConstrainedVisibleContentRect() const |
return viewportRect; |
} |
+bool FrameView::shouldPlaceVerticalScrollbarOnLeft() const |
+{ |
+ // Mainframe scrollbar placement should respect the embedding application RTL UI policy. |
Julien - ping for review
2014/02/12 23:25:49
That looks like a FIXME related to crbug.com/24986
|
+ if (isMainFrame()) |
+ return false; |
+ |
+ Document* document = m_frame->document(); |
+ if (!document) |
+ return false; |
+ |
+ if (!document->body() || !document->body()->renderer()) |
+ return false; |
+ |
+ // Use the <body> text direction to dictate the frame scrollbar placement. |
+ return document->body()->renderer()->style()->shouldPlaceBlockDirectionScrollbarOnLogicalLeft(); |
+} |
IntSize FrameView::scrollOffsetForFixedPosition() const |
{ |