Rename scrollPosition(Scrollbar*) to scrollOffset(ScrollbarOrientation)
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 3 Jan 2016 19:50:51 +0000 (19:50 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 3 Jan 2016 19:50:51 +0000 (19:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=152666

Reviewed by Dan Bernstein.

Scrollbar values take offsets (zero-based), so rename the accessor used
by scrollbars to set their value, and pass in an orientation, rather than
a pointer to the scrollbar.

Source/WebCore:

* platform/ScrollView.cpp:
(WebCore::ScrollView::scrollOffset): Use scrollOffsetFromPosition rather than
manually adding scrollOrigin.
(WebCore::ScrollView::scrollPosition): Renamed.
* platform/ScrollView.h:
(WebCore::ScrollView::cachedScrollPosition):
* platform/ScrollableArea.h:
* platform/Scrollbar.cpp:
(WebCore::Scrollbar::Scrollbar):
(WebCore::Scrollbar::offsetDidChange):
* rendering/RenderLayer.cpp:
(WebCore::RenderLayer::scrollOffset):
(WebCore::RenderLayer::scrollPosition): Renamed.
* rendering/RenderLayer.h:
* rendering/RenderListBox.cpp:
(WebCore::RenderListBox::scrollOffset):
(WebCore::RenderListBox::scrollPosition): Renamed.
* rendering/RenderListBox.h:

Source/WebKit2:

* WebProcess/Plugins/PDF/DeprecatedPDFPlugin.h:
* WebProcess/Plugins/PDF/DeprecatedPDFPlugin.mm:
(WebKit::PDFPlugin::scrollOffset):
(WebKit::PDFPlugin::scrollPosition): Renamed.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@194515 268f45cc-cd09-0410-ab3c-d52691b4dbfc

12 files changed:
Source/WebCore/ChangeLog
Source/WebCore/platform/ScrollView.cpp
Source/WebCore/platform/ScrollView.h
Source/WebCore/platform/ScrollableArea.h
Source/WebCore/platform/Scrollbar.cpp
Source/WebCore/rendering/RenderLayer.cpp
Source/WebCore/rendering/RenderLayer.h
Source/WebCore/rendering/RenderListBox.cpp
Source/WebCore/rendering/RenderListBox.h
Source/WebKit2/ChangeLog
Source/WebKit2/WebProcess/Plugins/PDF/DeprecatedPDFPlugin.h
Source/WebKit2/WebProcess/Plugins/PDF/DeprecatedPDFPlugin.mm

index bfba02b..f987b0d 100644 (file)
@@ -1,5 +1,35 @@
 2016-01-03  Simon Fraser  <simon.fraser@apple.com>
 
+        Rename scrollPosition(Scrollbar*) to scrollOffset(ScrollbarOrientation)
+        https://bugs.webkit.org/show_bug.cgi?id=152666
+
+        Reviewed by Dan Bernstein.
+
+        Scrollbar values take offsets (zero-based), so rename the accessor used
+        by scrollbars to set their value, and pass in an orientation, rather than
+        a pointer to the scrollbar.
+
+        * platform/ScrollView.cpp:
+        (WebCore::ScrollView::scrollOffset): Use scrollOffsetFromPosition rather than
+        manually adding scrollOrigin.
+        (WebCore::ScrollView::scrollPosition): Renamed.
+        * platform/ScrollView.h:
+        (WebCore::ScrollView::cachedScrollPosition):
+        * platform/ScrollableArea.h:
+        * platform/Scrollbar.cpp:
+        (WebCore::Scrollbar::Scrollbar):
+        (WebCore::Scrollbar::offsetDidChange):
+        * rendering/RenderLayer.cpp:
+        (WebCore::RenderLayer::scrollOffset):
+        (WebCore::RenderLayer::scrollPosition): Renamed.
+        * rendering/RenderLayer.h:
+        * rendering/RenderListBox.cpp:
+        (WebCore::RenderListBox::scrollOffset):
+        (WebCore::RenderListBox::scrollPosition): Renamed.
+        * rendering/RenderListBox.h:
+
+2016-01-03  Simon Fraser  <simon.fraser@apple.com>
+
         scrollLeft returns different values for viewport and non-viewport scrolling with rtl
         https://bugs.webkit.org/show_bug.cgi?id=94405
         rdar://problem/21829754
index bb79aba..c7c2f37 100644 (file)
@@ -508,12 +508,16 @@ void ScrollView::completeUpdatesAfterScrollTo(const IntSize& scrollDelta)
     updateCompositingLayersAfterScrolling();
 }
 
-int ScrollView::scrollPosition(Scrollbar* scrollbar) const
+int ScrollView::scrollOffset(ScrollbarOrientation orientation) const
 {
-    if (scrollbar->orientation() == HorizontalScrollbar)
-        return scrollPosition().x() + scrollOrigin().x();
-    if (scrollbar->orientation() == VerticalScrollbar)
-        return scrollPosition().y() + scrollOrigin().y();
+    ScrollOffset offset = scrollOffsetFromPosition(scrollPosition());
+
+    if (orientation == HorizontalScrollbar)
+        return offset.x();
+
+    if (orientation == VerticalScrollbar)
+        return offset.y();
+
     return 0;
 }
 
index c2690ef..7887fba 100644 (file)
@@ -66,7 +66,7 @@ public:
 
     // ScrollableArea functions.
     virtual int scrollSize(ScrollbarOrientation) const override;
-    virtual int scrollPosition(Scrollbar*) const override;
+    virtual int scrollOffset(ScrollbarOrientation) const override;
     WEBCORE_EXPORT virtual void setScrollOffset(const ScrollOffset&) override;
     virtual bool isScrollCornerVisible() const override;
     virtual void scrollbarStyleChanged(ScrollbarStyle, bool forceUpdate) override;
@@ -263,7 +263,7 @@ public:
     virtual IntSize overhangAmount() const override;
 
     void cacheCurrentScrollPosition() { m_cachedScrollPosition = scrollPosition(); }
-    IntPoint cachedScrollPosition() const { return m_cachedScrollPosition; }
+    ScrollPosition cachedScrollPosition() const { return m_cachedScrollPosition; }
 
     // Functions for scrolling the view.
     virtual void setScrollPosition(const ScrollPosition&);
index 4295d84..47bb274 100644 (file)
@@ -146,7 +146,7 @@ public:
 
     virtual bool isActive() const = 0;
     virtual int scrollSize(ScrollbarOrientation) const = 0;
-    virtual int scrollPosition(Scrollbar*) const = 0;
+    virtual int scrollOffset(ScrollbarOrientation) const = 0;
     WEBCORE_EXPORT virtual void invalidateScrollbar(Scrollbar*, const IntRect&);
     virtual bool isScrollCornerVisible() const = 0;
     virtual IntRect scrollCornerRect() const = 0;
@@ -281,10 +281,10 @@ public:
     void serviceScrollAnimations();
 
 #if PLATFORM(IOS)
-    bool isHorizontalScrollerPinnedToMinimumPosition() const { return !horizontalScrollbar() || scrollPosition(horizontalScrollbar()) <= minimumScrollPosition().x(); }
-    bool isHorizontalScrollerPinnedToMaximumPosition() const { return !horizontalScrollbar() || scrollPosition(horizontalScrollbar()) >= maximumScrollPosition().x(); }
-    bool isVerticalScrollerPinnedToMinimumPosition() const { return !verticalScrollbar() || scrollPosition(verticalScrollbar()) <= minimumScrollPosition().y(); }
-    bool isVerticalScrollerPinnedToMaximumPosition() const { return !verticalScrollbar() || scrollPosition(verticalScrollbar()) >= maximumScrollPosition().y(); } 
+    bool isHorizontalScrollerPinnedToMinimumPosition() const { return !horizontalScrollbar() || scrollOffset(HorizontalScrollbar) <= 0; }
+    bool isHorizontalScrollerPinnedToMaximumPosition() const { return !horizontalScrollbar() || scrollOffset(HorizontalScrollbar) >= maximumScrollOffset().x(); }
+    bool isVerticalScrollerPinnedToMinimumPosition() const { return !verticalScrollbar() || scrollOffset(VerticalScrollbar) <= 0; }
+    bool isVerticalScrollerPinnedToMaximumPosition() const { return !verticalScrollbar() || scrollOffset(VerticalScrollbar) >= maximumScrollOffset().y(); }
 
     bool isPinnedInBothDirections(const IntSize&) const; 
     bool isPinnedHorizontallyInDirection(int horizontalScrollDelta) const; 
index 8d8eff8..4b701c7 100644 (file)
@@ -87,7 +87,7 @@ Scrollbar::Scrollbar(ScrollableArea& scrollableArea, ScrollbarOrientation orient
     int thickness = theme().scrollbarThickness(controlSize);
     Widget::setFrameRect(IntRect(0, 0, thickness, thickness));
 
-    m_currentPos = static_cast<float>(m_scrollableArea.scrollPosition(this));
+    m_currentPos = static_cast<float>(m_scrollableArea.scrollOffset(m_orientation));
 }
 
 Scrollbar::~Scrollbar()
@@ -109,7 +109,7 @@ int Scrollbar::occupiedHeight() const
 
 void Scrollbar::offsetDidChange()
 {
-    float position = static_cast<float>(m_scrollableArea.scrollPosition(this));
+    float position = static_cast<float>(m_scrollableArea.scrollOffset(m_orientation));
     if (position == m_currentPos)
         return;
 
index 0019fe5..bdfed53 100644 (file)
@@ -2735,12 +2735,14 @@ void RenderLayer::setScrollOffset(const ScrollOffset& offset)
     scrollTo(scrollPositionFromOffset(offset));
 }
 
-int RenderLayer::scrollPosition(Scrollbar* scrollbar) const
+int RenderLayer::scrollOffset(ScrollbarOrientation orientation) const
 {
-    if (scrollbar->orientation() == HorizontalScrollbar)
+    if (orientation == HorizontalScrollbar)
         return scrollOffset().x();
-    if (scrollbar->orientation() == VerticalScrollbar)
+
+    if (orientation == VerticalScrollbar)
         return scrollOffset().y();
+
     return 0;
 }
 
index 380509e..bd48123 100644 (file)
@@ -855,7 +855,7 @@ private:
 
     bool shouldBeSelfPaintingLayer() const;
 
-    virtual int scrollPosition(Scrollbar*) const override;
+    virtual int scrollOffset(ScrollbarOrientation) const override;
     
     // ScrollableArea interface
     virtual void invalidateScrollbarRect(Scrollbar*, const IntRect&) override;
index 764539b..0e7c428 100644 (file)
@@ -603,7 +603,7 @@ int RenderListBox::scrollSize(ScrollbarOrientation orientation) const
     return ((orientation == VerticalScrollbar) && m_vBar) ? (m_vBar->totalSize() - m_vBar->visibleSize()) : 0;
 }
 
-int RenderListBox::scrollPosition(Scrollbar*) const
+int RenderListBox::scrollOffset(ScrollbarOrientation) const
 {
     return m_indexOffset;
 }
index 2f8697a..a2b0492 100644 (file)
@@ -110,7 +110,7 @@ private:
 
     // ScrollableArea interface.
     virtual int scrollSize(ScrollbarOrientation) const override;
-    virtual int scrollPosition(Scrollbar*) const override;
+    virtual int scrollOffset(ScrollbarOrientation) const override;
     virtual void setScrollOffset(const ScrollOffset&) override;
     virtual ScrollPosition minimumScrollPosition() const override;
     virtual ScrollPosition maximumScrollPosition() const override;
index ff4204c..a688f42 100644 (file)
@@ -1,3 +1,19 @@
+2016-01-03  Simon Fraser  <simon.fraser@apple.com>
+
+        Rename scrollPosition(Scrollbar*) to scrollOffset(ScrollbarOrientation)
+        https://bugs.webkit.org/show_bug.cgi?id=152666
+
+        Reviewed by Dan Bernstein.
+
+        Scrollbar values take offsets (zero-based), so rename the accessor used
+        by scrollbars to set their value, and pass in an orientation, rather than
+        a pointer to the scrollbar.
+
+        * WebProcess/Plugins/PDF/DeprecatedPDFPlugin.h:
+        * WebProcess/Plugins/PDF/DeprecatedPDFPlugin.mm:
+        (WebKit::PDFPlugin::scrollOffset):
+        (WebKit::PDFPlugin::scrollPosition): Renamed.
+
 2015-12-31  Andy Estes  <aestes@apple.com>
 
         Replace WTF::move with WTFMove
index 17334d9..30caf66 100644 (file)
@@ -196,7 +196,7 @@ private:
     virtual int scrollSize(WebCore::ScrollbarOrientation) const override;
     virtual bool isActive() const override;
     virtual bool isScrollCornerVisible() const override { return false; }
-    virtual int scrollPosition(WebCore::Scrollbar*) const override;
+    virtual int scrollOffset(WebCore::ScrollbarOrientation) const override;
     virtual WebCore::ScrollPosition scrollPosition() const override;
     virtual WebCore::ScrollPosition minimumScrollPosition() const override;
     virtual WebCore::ScrollPosition maximumScrollPosition() const override;
index a12d3ff..551ef19 100644 (file)
@@ -773,12 +773,14 @@ bool PDFPlugin::forceUpdateScrollbarsOnMainThreadForPerformanceTesting() const
     return false;
 }
 
-int PDFPlugin::scrollPosition(Scrollbar* scrollbar) const
+int PDFPlugin::scrollOffset(ScrollbarOrientation orientation) const
 {
-    if (scrollbar->orientation() == HorizontalScrollbar)
+    if (orientation == HorizontalScrollbar)
         return m_scrollOffset.width();
-    if (scrollbar->orientation() == VerticalScrollbar)
+
+    if (orientation == VerticalScrollbar)
         return m_scrollOffset.height();
+
     ASSERT_NOT_REACHED();
     return 0;
 }