Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1494)

Unified Diff: Source/core/rendering/RenderListBox.cpp

Issue 23480037: Fix RTL scroll bars being misrendered on Android. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: isLeftSideVerticalScrollbar Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/rendering/RenderListBox.cpp
diff --git a/Source/core/rendering/RenderListBox.cpp b/Source/core/rendering/RenderListBox.cpp
index f13ef693c7c0b82ba0371417980af2d2574dea88..ea88d153db5b01f24775716bf724e9b9df03ecef 100644
--- a/Source/core/rendering/RenderListBox.cpp
+++ b/Source/core/rendering/RenderListBox.cpp
@@ -276,8 +276,8 @@ int RenderListBox::baselinePosition(FontBaseline baselineType, bool firstLine, L
LayoutRect RenderListBox::itemBoundingBoxRect(const LayoutPoint& additionalOffset, int index)
{
return LayoutRect(additionalOffset.x() + borderLeft() + paddingLeft(),
- additionalOffset.y() + borderTop() + paddingTop() + itemHeight() * (index - m_indexOffset),
- contentWidth(), itemHeight());
+ additionalOffset.y() + borderTop() + paddingTop() + itemHeight() * (index - m_indexOffset),
+ contentWidth(), itemHeight());
}
void RenderListBox::paintObject(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
@@ -843,6 +843,11 @@ bool RenderListBox::userInputScrollable(ScrollbarOrientation orientation) const
return orientation == VerticalScrollbar;
}
+bool RenderListBox::shouldPlaceVerticalScrollbarOnLeft() const
+{
+ return false;
+}
+
int RenderListBox::lineStep(ScrollbarOrientation) const
{
return 1;

Powered by Google App Engine
This is Rietveld 408576698