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

Unified Diff: Source/WebKit/chromium/src/ScrollbarGroup.cpp

Issue 16982005: Allow objects without scrollbars to be scrollable (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: CR fixes Created 7 years, 6 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/WebKit/chromium/src/ScrollbarGroup.cpp
diff --git a/Source/WebKit/chromium/src/ScrollbarGroup.cpp b/Source/WebKit/chromium/src/ScrollbarGroup.cpp
index 3fa766eb82ec47b613fa053f8b428a105ef5bd5e..39f5780a1f67a33f16964408aac14d3d1e399ad9 100644
--- a/Source/WebKit/chromium/src/ScrollbarGroup.cpp
+++ b/Source/WebKit/chromium/src/ScrollbarGroup.cpp
@@ -100,14 +100,6 @@ int ScrollbarGroup::scrollSize(WebCore::ScrollbarOrientation orientation) const
return scrollbar->totalSize() - scrollbar->visibleSize();
}
-int ScrollbarGroup::scrollPosition(Scrollbar* scrollbar) const
-{
- WebPluginScrollbarImpl* webScrollbar = scrollbar->orientation() == HorizontalScrollbar ? m_horizontalScrollbar : m_verticalScrollbar;
- if (!webScrollbar)
- return 0;
- return webScrollbar->scrollOffset();
-}
-
void ScrollbarGroup::setScrollOffset(const IntPoint& offset)
{
if (m_horizontalScrollbar && m_horizontalScrollbar->scrollOffset() != offset.x())
@@ -268,4 +260,14 @@ bool ScrollbarGroup::scrollbarsCanBeActive() const
return true;
}
+bool ScrollbarGroup::userInputScrollable(ScrollbarOrientation orientation) const
+{
+ return orientation == HorizontalScrollbar ? horizontalScrollbar() : verticalScrollbar();
+}
+
+int ScrollbarGroup::pageStep(ScrollbarOrientation orientation) const
+{
+ return 0;
+}
+
} // namespace WebKit

Powered by Google App Engine
This is Rietveld 408576698