Extract touch handling logic from EventHandler

third_party/WebKit/Source/core/events/PointerEventFactory.h

Index: third_party/WebKit/Source/core/events/PointerEventFactory.h
diff --git a/third_party/WebKit/Source/core/events/PointerEventFactory.h b/third_party/WebKit/Source/core/events/PointerEventFactory.h
index aa155ce53c2994b99ee36f3fb9dc444e0ea75ddf..7c96f41c3230e0393b1432663af23916254c6644 100644
--- a/third_party/WebKit/Source/core/events/PointerEventFactory.h
+++ b/third_party/WebKit/Source/core/events/PointerEventFactory.h
@@ -36,8 +36,8 @@ public:
 
     PointerEvent* create(const AtomicString& type,
         const PlatformTouchPoint&, PlatformEvent::Modifiers,
-        const double width, const double height,
-        const double clientX, const double clientY);
+        const FloatSize& pointRadius,
+        const FloatPoint& pagePoint);

[tdresser, 2016/04/29 15:55:04]

client != page: was this a semantic change?

[Navid Zolghadr, 2016/04/29 16:31:19]

I don't know much about this one. I just picked the name of the parameter we
were passing before and put it as the name of the parameter. Although I
understand that we use it in setClientX/Y apis. I'll double check this with
Bokan when I add him to this review to see which one should be changed.

 
     PointerEvent* createPointerCancelEvent(
         const int pointerId, const WebPointerProperties::PointerType);
@@ -62,16 +62,20 @@ public:
     bool remove(const int);
 
     // Returns all ids of the given pointerType.
-    Vector<int> getPointerIdsOfType(WebPointerProperties::PointerType);
+    Vector<int> getPointerIdsOfType(WebPointerProperties::PointerType) const;
 
-    // Returns whether a pointer id exists and active
-    bool isActive(const int);
+    // Returns whether a pointer id exists and active.
+    bool isActive(const int) const;
 
-    // Returns type of pointer id if exists, otherwise Unknown
-    WebPointerProperties::PointerType getPointerType(const int);
+    // Returns type of pointer id if exists, otherwise Unknown.
+    WebPointerProperties::PointerType getPointerType(const int) const;
 
-    // Returns whether a pointer id exists and has at least one pressed button
-    bool isActiveButtonsState(const int);
+    // Returns whether a pointer id exists and has at least one pressed button.
+    bool isActiveButtonsState(const int) const;
+
+    // Returns the id of the pointer event corresponding to the given pointer
+    // properties if exists otherwise s_invalidId.
+    int getPointerEventId(const WebPointerProperties&) const;
 
 private:
     typedef WTF::UnsignedWithZeroKeyHashTraits<int> UnsignedHash;