Move touch hit testing to PointerEventManager

third_party/WebKit/Source/core/input/TouchEventManager.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef TouchEventManager_h
 #define TouchEventManager_h
 
 #include "core/CoreExport.h"
 #include "core/events/PointerEventFactory.h"
 #include "platform/UserGestureIndicator.h"
@@ skipping 22 matching lines @@
             visitor->trace(touchNode);
             visitor->trace(targetFrame);
         }
 
         PlatformTouchPoint point;
         Member<Node> touchNode;
         Member<LocalFrame> targetFrame;
         FloatPoint adjustedPagePoint;
         FloatSize adjustedRadius;
         bool knownTarget;
-        bool consumed;

[mustaq, 2016/05/11 14:30:55]

I don't think we can afford to lose this until crbug.com/607588 is fixed. You
can either bring it back, or delay this CL until after the other bug is fixed.

[Navid Zolghadr, 2016/05/11 16:43:42]

Let's discuss this offline in more details. But I believe this should cause any
difficulty for that. This CL just uses another mechanism for preventing the
touch event and bypassing that will be easy. That mechanism lives in
PointerEventManager.

         String region;
     };
 
     explicit TouchEventManager(LocalFrame*);
     ~TouchEventManager();
     DECLARE_TRACE();
 
-    // Returns true if it succesfully generates touchInfos.
-    bool generateTouchInfosAfterHittest(
+    // Does the hit-testing again if the original hit test result was not inside
+    // capturing frame for touch events.
+    bool reHitTestTouchPointsIfNeeded(
         const PlatformTouchEvent&,
         HeapVector<TouchInfo>&);
 
     WebInputEventResult handleTouchEvent(
         const PlatformTouchEvent&,
-        const HeapVector<TouchInfo>&);
+        HeapVector<TouchInfo>&);
 
     // Resets the internal state of this object.
     void clear();
 
     // Returns whether there is any touch on the screen.
     bool isAnyTouchActive() const;
 
 private:
 
     void updateTargetAndRegionMapsForTouchStarts(HeapVector<TouchInfo>&);
@@ skipping 24 matching lines @@
     // True if waiting on first touch move after a touch start.
     bool m_waitingForFirstTouchMove;
 
 };
 
 } // namespace blink
 
 WTF_ALLOW_INIT_WITH_MEM_FUNCTIONS(blink::TouchEventManager::TouchInfo);
 
 #endif // TouchEventManager_h