Title: [271564] trunk/Source/WebCore
Revision
271564
Author
[email protected]
Date
2021-01-17 13:06:49 -0800 (Sun, 17 Jan 2021)

Log Message

Unreviewed, fix the internal iOS build after r271559

Fix the build with IOS_TOUCH_EVENTS enabled.

* rendering/RenderLayer.cpp:
(WebCore::RenderLayer::handleTouchEvent): Deleted.
(WebCore::RenderLayer::registerAsTouchEventListenerForScrolling): Deleted.
(WebCore::RenderLayer::unregisterAsTouchEventListenerForScrolling): Deleted.

Remove a few undeclared method definitions. These methods were relocated to `RenderLayerScrollableArea`.

* rendering/RenderLayerScrollableArea.cpp:

Remove several methods that have duplicate implementations.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (271563 => 271564)


--- trunk/Source/WebCore/ChangeLog	2021-01-17 19:55:10 UTC (rev 271563)
+++ trunk/Source/WebCore/ChangeLog	2021-01-17 21:06:49 UTC (rev 271564)
@@ -1,3 +1,20 @@
+2021-01-17  Wenson Hsieh  <[email protected]>
+
+        Unreviewed, fix the internal iOS build after r271559
+
+        Fix the build with IOS_TOUCH_EVENTS enabled.
+
+        * rendering/RenderLayer.cpp:
+        (WebCore::RenderLayer::handleTouchEvent): Deleted.
+        (WebCore::RenderLayer::registerAsTouchEventListenerForScrolling): Deleted.
+        (WebCore::RenderLayer::unregisterAsTouchEventListenerForScrolling): Deleted.
+
+        Remove a few undeclared method definitions. These methods were relocated to `RenderLayerScrollableArea`.
+
+        * rendering/RenderLayerScrollableArea.cpp:
+
+        Remove several methods that have duplicate implementations.
+
 2021-01-17  Zalan Bujtas  <[email protected]>
 
         [LFC][IFC] No need to maintain the "is considered empty" bit anymore.

Modified: trunk/Source/WebCore/rendering/RenderLayer.cpp (271563 => 271564)


--- trunk/Source/WebCore/rendering/RenderLayer.cpp	2021-01-17 19:55:10 UTC (rev 271563)
+++ trunk/Source/WebCore/rendering/RenderLayer.cpp	2021-01-17 21:06:49 UTC (rev 271564)
@@ -2583,35 +2583,6 @@
     return toLayoutSize(convertToLayerCoords(ancestorLayer, LayoutPoint(), adjustForColumns));
 }
 
-#if ENABLE(IOS_TOUCH_EVENTS)
-bool RenderLayer::handleTouchEvent(const PlatformTouchEvent& touchEvent)
-{
-    // If we have accelerated scrolling, let the scrolling be handled outside of WebKit.
-    if (hasCompositedScrollableOverflow())
-        return false;
-
-    return ScrollableArea::handleTouchEvent(touchEvent);
-}
-
-void RenderLayer::registerAsTouchEventListenerForScrolling()
-{
-    if (!renderer().element() || m_registeredAsTouchEventListenerForScrolling)
-        return;
-    
-    renderer().document().addTouchEventHandler(*renderer().element());
-    m_registeredAsTouchEventListenerForScrolling = true;
-}
-
-void RenderLayer::unregisterAsTouchEventListenerForScrolling()
-{
-    if (!renderer().element() || !m_registeredAsTouchEventListenerForScrolling)
-        return;
-
-    renderer().document().removeTouchEventHandler(*renderer().element());
-    m_registeredAsTouchEventListenerForScrolling = false;
-}
-#endif // ENABLE(IOS_TOUCH_EVENTS)
-
 static inline bool frameElementAndViewPermitScroll(HTMLFrameElementBase* frameElementBase, FrameView& frameView)
 {
     // If scrollbars aren't explicitly forbidden, permit scrolling.

Modified: trunk/Source/WebCore/rendering/RenderLayerScrollableArea.cpp (271563 => 271564)


--- trunk/Source/WebCore/rendering/RenderLayerScrollableArea.cpp	2021-01-17 19:55:10 UTC (rev 271563)
+++ trunk/Source/WebCore/rendering/RenderLayerScrollableArea.cpp	2021-01-17 21:06:49 UTC (rev 271564)
@@ -1638,37 +1638,6 @@
     updateResizerStyle();
 }
 
-#if ENABLE(IOS_TOUCH_EVENTS)
-bool RenderLayerScrollableArea::handleTouchEvent(const PlatformTouchEvent& touchEvent)
-{
-    // If we have accelerated scrolling, let the scrolling be handled outside of WebKit.
-    if (hasCompositedScrollableOverflow())
-        return false;
-
-    return ScrollableArea::handleTouchEvent(touchEvent);
-}
-
-void RenderLayerScrollableArea::registerAsTouchEventListenerForScrolling()
-{
-    auto& renderer = m_layer.renderer();
-    if (!renderer.element() || m_registeredAsTouchEventListenerForScrolling)
-        return;
-
-    renderer.document().addTouchEventHandler(*renderer.element());
-    m_registeredAsTouchEventListenerForScrolling = true;
-}
-
-void RenderLayerScrollableArea::unregisterAsTouchEventListenerForScrolling()
-{
-    auto& renderer = m_layer.renderer;
-    if (!renderer.element() || !m_registeredAsTouchEventListenerForScrolling)
-        return;
-
-    renderer.document().removeTouchEventHandler(*renderer.element());
-    m_registeredAsTouchEventListenerForScrolling = false;
-}
-#endif // ENABLE(IOS_TOUCH_EVENTS)
-
 // FIXME: this is only valid after we've made layers.
 bool RenderLayerScrollableArea::usesCompositedScrolling() const
 {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to