Allow objects without scrollbars to be scrollable

Source/web/ScrollbarGroup.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 82 matching lines @@
 
 int ScrollbarGroup::scrollSize(WebCore::ScrollbarOrientation orientation) const
 {
     WebPluginScrollbarImpl* webScrollbar = orientation == HorizontalScrollbar ? m_horizontalScrollbar : m_verticalScrollbar;
     if (!webScrollbar)
         return 0;
     Scrollbar* scrollbar = webScrollbar->scrollbar();
     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())
         m_horizontalScrollbar->setScrollOffset(offset.x());
     else if (m_verticalScrollbar && m_verticalScrollbar->scrollOffset() != offset.y())
         m_verticalScrollbar->setScrollOffset(offset.y());
 }
 
 void ScrollbarGroup::invalidateScrollbarRect(Scrollbar* scrollbar, const IntRect& rect)
 {
@@ skipping 140 matching lines @@
         m_horizontalScrollbar->scrollbarStyleChanged();
     if (m_verticalScrollbar)
         m_verticalScrollbar->scrollbarStyleChanged();
 }
 
 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