Conditional hittesting & sending boundary events for captured pointer events

third_party/WebKit/Source/core/input/EventHandler.cpp

Index: third_party/WebKit/Source/core/input/EventHandler.cpp
diff --git a/third_party/WebKit/Source/core/input/EventHandler.cpp b/third_party/WebKit/Source/core/input/EventHandler.cpp
index 34af1394795db9d870cd30cf955df3490c0420d5..7698daaf1bf4a4dabb5632e62002e3c850e1639c 100644
--- a/third_party/WebKit/Source/core/input/EventHandler.cpp
+++ b/third_party/WebKit/Source/core/input/EventHandler.cpp
@@ -1384,7 +1384,9 @@ Node* EventHandler::updateMouseEventTargetNode(Node* targetNode,
     Node* newNodeUnderMouse = targetNode;
 
     // If we're capturing, we always go right to that node.
-    if (EventTarget* mousePointerCapturingNode = m_pointerEventManager->getMouseCapturingNode()) {

[dtapuska, 2016/09/09 16:01:07]

I think it is cleaner to store the mousePointerCapturingNode in a variable
instead of calling it multiple times for ease of reading.

[Navid Zolghadr, 2016/09/09 16:03:38]

Then I have to store it outside the scope of the if which makes it defined for
the whole function scope. Is there a way to do it better?

+    if (m_pointerEventManager->getMouseCapturingNode()
+        && !RuntimeEnabledFeatures::pointerEventV1SpecCapturingEnabled()) {
+        EventTarget* mousePointerCapturingNode = m_pointerEventManager->getMouseCapturingNode();
         newNodeUnderMouse = mousePointerCapturingNode->toNode();
         DCHECK(newNodeUnderMouse);
     } else if (m_capturingMouseEventsNode) {