Remove unused fields from Scrollbar
authorantti@apple.com <antti@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 Jan 2019 15:46:19 +0000 (15:46 +0000)
committerantti@apple.com <antti@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 15 Jan 2019 15:46:19 +0000 (15:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=193442

Reviewed by Zalan Bujtas.

* platform/Scrollbar.cpp:
(WebCore::Scrollbar::Scrollbar):
* platform/Scrollbar.h:
(WebCore::Scrollbar::isCustomScrollbar const):

Make virtual so it doesn't need a bit.

(WebCore::Scrollbar::isAlphaLocked const): Deleted.
(WebCore::Scrollbar::setIsAlphaLocked): Deleted.
* platform/mac/ScrollAnimatorMac.mm:
(WebCore::ScrollAnimatorMac::shouldScrollbarParticipateInHitTesting):
* rendering/RenderScrollbar.cpp:
(WebCore::RenderScrollbar::RenderScrollbar):
* rendering/RenderScrollbar.h:

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

Source/WebCore/ChangeLog
Source/WebCore/platform/Scrollbar.cpp
Source/WebCore/platform/Scrollbar.h
Source/WebCore/platform/mac/ScrollAnimatorMac.mm
Source/WebCore/rendering/RenderScrollbar.cpp
Source/WebCore/rendering/RenderScrollbar.h

index c3f185d..3f5e1f5 100644 (file)
@@ -1,3 +1,25 @@
+2019-01-15  Antti Koivisto  <antti@apple.com>
+
+        Remove unused fields from Scrollbar
+        https://bugs.webkit.org/show_bug.cgi?id=193442
+
+        Reviewed by Zalan Bujtas.
+
+        * platform/Scrollbar.cpp:
+        (WebCore::Scrollbar::Scrollbar):
+        * platform/Scrollbar.h:
+        (WebCore::Scrollbar::isCustomScrollbar const):
+
+        Make virtual so it doesn't need a bit.
+
+        (WebCore::Scrollbar::isAlphaLocked const): Deleted.
+        (WebCore::Scrollbar::setIsAlphaLocked): Deleted.
+        * platform/mac/ScrollAnimatorMac.mm:
+        (WebCore::ScrollAnimatorMac::shouldScrollbarParticipateInHitTesting):
+        * rendering/RenderScrollbar.cpp:
+        (WebCore::RenderScrollbar::RenderScrollbar):
+        * rendering/RenderScrollbar.h:
+
 2019-01-15  Zalan Bujtas  <zalan@apple.com>
 
         [LFC] Use the containing block's padding box to position out-of-flow elements.
index 39caa2f..25f8af1 100644 (file)
@@ -54,29 +54,12 @@ int Scrollbar::maxOverlapBetweenPages()
     return maxOverlapBetweenPages;
 }
 
-Scrollbar::Scrollbar(ScrollableArea& scrollableArea, ScrollbarOrientation orientation, ScrollbarControlSize controlSize, ScrollbarTheme* customTheme, bool isCustomScrollbar)
+Scrollbar::Scrollbar(ScrollableArea& scrollableArea, ScrollbarOrientation orientation, ScrollbarControlSize controlSize, ScrollbarTheme* customTheme)
     : m_scrollableArea(scrollableArea)
     , m_orientation(orientation)
     , m_controlSize(controlSize)
     , m_theme(customTheme ? *customTheme : ScrollbarTheme::theme())
-    , m_visibleSize(0)
-    , m_totalSize(0)
-    , m_currentPos(0)
-    , m_dragOrigin(0)
-    , m_lineStep(0)
-    , m_pageStep(0)
-    , m_pixelStep(1)
-    , m_hoveredPart(NoPart)
-    , m_pressedPart(NoPart)
-    , m_pressedPos(0)
-    , m_scrollPos(0)
-    , m_draggingDocument(false)
-    , m_documentDragPos(0)
-    , m_enabled(true)
     , m_scrollTimer(*this, &Scrollbar::autoscrollTimerFired)
-    , m_suppressInvalidation(false)
-    , m_isAlphaLocked(false)
-    , m_isCustomScrollbar(isCustomScrollbar)
 {
     theme().registerScrollbar(*this);
 
index f5cbe9e..c2bc6c2 100644 (file)
@@ -56,7 +56,7 @@ public:
 
     ScrollableArea& scrollableArea() const { return m_scrollableArea; }
 
-    bool isCustomScrollbar() const { return m_isCustomScrollbar; }
+    virtual bool isCustomScrollbar() const { return false; }
     ScrollbarOrientation orientation() const { return m_orientation; }
 
     int value() const { return lroundf(m_currentPos); }
@@ -124,16 +124,15 @@ public:
 
     void moveThumb(int pos, bool draggingDocument = false);
 
-    bool isAlphaLocked() const { return m_isAlphaLocked; }
-    void setIsAlphaLocked(bool flag) { m_isAlphaLocked = flag; }
-
+#if !PLATFORM(COCOA)
     float opacity() const { return m_opacity; }
     void setOpacity(float opacity) { m_opacity = opacity; }
+#endif
 
     bool supportsUpdateOnSecondaryThread() const;
 
 protected:
-    Scrollbar(ScrollableArea&, ScrollbarOrientation, ScrollbarControlSize, ScrollbarTheme* = 0, bool isCustomScrollbar = false);
+    Scrollbar(ScrollableArea&, ScrollbarOrientation, ScrollbarControlSize, ScrollbarTheme* = nullptr);
 
     void updateThumb();
     virtual void updateThumbPosition();
@@ -151,32 +150,29 @@ protected:
     ScrollbarControlSize m_controlSize;
     ScrollbarTheme& m_theme;
 
-    int m_visibleSize;
-    int m_totalSize;
-    float m_currentPos;
-    float m_dragOrigin;
-    int m_lineStep;
-    int m_pageStep;
-    float m_pixelStep;
+    int m_visibleSize { 0 };
+    int m_totalSize { 0 };
+    float m_currentPos { 0 };
+    float m_dragOrigin { 0 };
+    int m_lineStep { 0 };
+    int m_pageStep { 0 };
+    float m_pixelStep { 1 };
 
-    ScrollbarPart m_hoveredPart;
-    ScrollbarPart m_pressedPart;
-    int m_pressedPos;
-    float m_scrollPos;
-    bool m_draggingDocument;
-    int m_documentDragPos;
+    ScrollbarPart m_hoveredPart { NoPart };
+    ScrollbarPart m_pressedPart { NoPart };
+    int m_pressedPos { 0 };
+    bool m_draggingDocument { false };
+    int m_documentDragPos { 0 };
 
-    bool m_enabled;
+    bool m_enabled { true };
 
     Timer m_scrollTimer;
 
-    bool m_suppressInvalidation;
-
-    bool m_isAlphaLocked;
-
-    bool m_isCustomScrollbar;
+    bool m_suppressInvalidation { false };
 
+#if !PLATFORM(COCOA)
     float m_opacity { 1 };
+#endif
 
 private:
     bool isScrollbar() const override { return true; }
index e0cf532..7dac292 100644 (file)
@@ -1125,9 +1125,6 @@ bool ScrollAnimatorMac::shouldScrollbarParticipateInHitTesting(Scrollbar* scroll
     if (ScrollerStyle::recommendedScrollerStyle() != NSScrollerStyleOverlay)
         return true;
 
-    if (scrollbar->isAlphaLocked())
-        return true;
-
     // Overlay scrollbars should participate in hit testing whenever they are at all visible.
     NSScrollerImp *painter = scrollerImpForScrollbar(*scrollbar);
     if (!painter)
index 422ba92..308d237 100644 (file)
@@ -42,7 +42,7 @@ Ref<Scrollbar> RenderScrollbar::createCustomScrollbar(ScrollableArea& scrollable
 }
 
 RenderScrollbar::RenderScrollbar(ScrollableArea& scrollableArea, ScrollbarOrientation orientation, Element* ownerElement, Frame* owningFrame)
-    : Scrollbar(scrollableArea, orientation, RegularScrollbar, RenderScrollbarTheme::renderScrollbarTheme(), true)
+    : Scrollbar(scrollableArea, orientation, RegularScrollbar, RenderScrollbarTheme::renderScrollbarTheme())
     , m_ownerElement(ownerElement)
     , m_owningFrame(owningFrame)
 {
index 176fcca..e585605 100644 (file)
@@ -54,8 +54,6 @@ public:
 
     int minimumThumbLength();
 
-    bool isOverlayScrollbar() const override { return false; }
-
     float opacity();
 
     std::unique_ptr<RenderStyle> getScrollbarPseudoStyle(ScrollbarPart, PseudoId);
@@ -63,6 +61,9 @@ public:
 private:
     RenderScrollbar(ScrollableArea&, ScrollbarOrientation, Element*, Frame*);
 
+    bool isCustomScrollbar() const override { return true; }
+    bool isOverlayScrollbar() const override { return false; }
+
     void setParent(ScrollView*) override;
     void setEnabled(bool) override;