Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1071)

Unified Diff: Source/core/frame/EventHandlerRegistry.cpp

Issue 1055683003: (NOT FOR REVIEW) Distinguish between touch and touchmove handler presence (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fix loading Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/frame/EventHandlerRegistry.cpp
diff --git a/Source/core/frame/EventHandlerRegistry.cpp b/Source/core/frame/EventHandlerRegistry.cpp
index 8c3098603fd1650e68f676ec24e1f0fdeb5d8720..c77f1c7b0a452825756c0199ece93fe5052b623a 100644
--- a/Source/core/frame/EventHandlerRegistry.cpp
+++ b/Source/core/frame/EventHandlerRegistry.cpp
@@ -56,6 +56,8 @@ bool EventHandlerRegistry::eventTypeToClass(const AtomicString& eventType, Event
*result = ScrollEvent;
} else if (eventType == EventTypeNames::wheel || eventType == EventTypeNames::mousewheel) {
*result = WheelEvent;
+ } else if (eventType == EventTypeNames::touchmove || eventType == EventTypeNames::pointermove) {
+ *result = TouchMoveEvent;
} else if (isTouchEventType(eventType)) {
*result = TouchEvent;
} else if (isPointerEventType(eventType)) {
@@ -210,6 +212,9 @@ void EventHandlerRegistry::notifyHasHandlersChanged(EventHandlerClass handlerCla
case TouchEvent:
m_frameHost->chromeClient().needTouchEvents(hasActiveHandlers);
break;
+ case TouchMoveEvent:
+ m_frameHost->chromeClient().needTouchMoveEvents(hasActiveHandlers);
+ break;
#if ENABLE(ASSERT)
case EventsForTesting:
break;
@@ -223,7 +228,7 @@ void EventHandlerRegistry::notifyHasHandlersChanged(EventHandlerClass handlerCla
void EventHandlerRegistry::notifyDidAddOrRemoveEventHandlerTarget(EventHandlerClass handlerClass)
{
ScrollingCoordinator* scrollingCoordinator = m_frameHost->page().scrollingCoordinator();
- if (scrollingCoordinator && handlerClass == TouchEvent)
+ if (scrollingCoordinator && (handlerClass == TouchEvent || handlerClass == TouchMoveEvent))
scrollingCoordinator->touchEventTargetRectsDidChange();
}

Powered by Google App Engine
This is Rietveld 408576698