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

Unified Diff: third_party/WebKit/Source/core/input/PointerEventManager.cpp

Issue 2408133007: Check the root frame pointer events as well for capture (Closed)
Patch Set: Only lookup the root frame for touch pointer events Created 4 years, 2 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: third_party/WebKit/Source/core/input/PointerEventManager.cpp
diff --git a/third_party/WebKit/Source/core/input/PointerEventManager.cpp b/third_party/WebKit/Source/core/input/PointerEventManager.cpp
index 21ce995d0f67a222d0160e4a6e62a6fa861c49a5..a99569be6315bdde46c84228c01f41fa43d1b754 100644
--- a/third_party/WebKit/Source/core/input/PointerEventManager.cpp
+++ b/third_party/WebKit/Source/core/input/PointerEventManager.cpp
@@ -655,6 +655,20 @@ bool PointerEventManager::isActive(const int pointerId) const {
return m_pointerEventFactory.isActive(pointerId);
}
+bool PointerEventManager::isTouchPointerIdActiveOnFrame(
+ int pointerId,
+ LocalFrame* frame) const {
+ if (m_pointerEventFactory.getPointerType(pointerId) !=
mustaq 2016/10/17 20:18:56 Please add a comment about why this is Touch-only.
Navid Zolghadr 2016/10/18 14:18:11 Done.
+ WebPointerProperties::PointerType::Touch)
+ return false;
+ Node* lastNodeReceivingEvent =
+ m_nodeUnderPointer.contains(pointerId)
+ ? m_nodeUnderPointer.get(pointerId).target->toNode()
+ : nullptr;
+ return lastNodeReceivingEvent &&
+ lastNodeReceivingEvent->document().frame() == frame;
+}
+
bool PointerEventManager::isAnyTouchActive() const {
return m_touchEventManager->isAnyTouchActive();
}

Powered by Google App Engine
This is Rietveld 408576698