Allow objects without scrollbars to be scrollable

Source/web/ScrollbarGroup.cpp

Index: Source/web/ScrollbarGroup.cpp
diff --git a/Source/web/ScrollbarGroup.cpp b/Source/web/ScrollbarGroup.cpp
index 3fa766eb82ec47b613fa053f8b428a105ef5bd5e..39f5780a1f67a33f16964408aac14d3d1e399ad9 100644
--- a/Source/web/ScrollbarGroup.cpp
+++ b/Source/web/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;

[jamesr, 2013/07/31 22:44:48]

this doesn't look right.  have you tested page up/down in a pdf document
rendered by chrome's pdf viewer?

[bokan, 2013/08/01 19:08:31]

This was an oversight. Fixed and tested in pdf viewer.

+}
+
 } // namespace WebKit