Pointerevent capture APIs

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

 // Copyright 2015 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 PointerEventManager_h
 #define PointerEventManager_h
 
 #include "core/CoreExport.h"
 #include "core/events/PointerEvent.h"
 #include "public/platform/WebPointerProperties.h"
@@ skipping 35 matching lines @@
     /*
      * When a pointerEvent with a particular id is removed that id is considered
      * free even though there might have been other PointerEvents that were
      * generated with the same id before.
      */
     void remove(const PassRefPtrWillBeRawPtr<PointerEvent>);
 
     // Returns the node capturing this pointer id, and null if no node is capturing it.
     EventTarget* getCapturingNode(PassRefPtrWillBeRawPtr<PointerEvent>);
 
+    void elementRemoved(EventTarget*);
+    void setPointerCapture(int, EventTarget*);
+    void releasePointerCapture(int, EventTarget*);
+    void implicitReleasePointerCapture(int);
+    bool isActive(const int);
+    EventTarget* getMouseCapturingNode();
+
 private:
-    typedef long MappedId;
     typedef std::pair<int, int> IncomingId;
+    typedef WillBeHeapHashMap<int, RefPtrWillBeMember<EventTarget>> PointerCapturingMap;
 
-    MappedId add(const IncomingId);
-    bool isPrimary(const MappedId) const;
-    void setIdAndType(PointerEventInit &, const WebPointerProperties &);
+    int addAndUpdateButtons(const IncomingId, const int);
+    int add(const IncomingId);
+    bool isPrimary(const int) const;
+    void setIdAndType(PointerEventInit &, const WebPointerProperties &, int);
+    bool isActiveButtons(const int);
+    void processPendingPointerCapture(int);
+    void removeTargetFromPointerCapturingMapping(PointerCapturingMap&, EventTarget*);
 
-    static const MappedId s_invalidId;
-    static const MappedId s_mouseId;
+    static const int s_invalidId;
+    static const int s_mouseId;
 
-    PointerEventManager::MappedId m_currentId;
-    HashMap<IncomingId, MappedId, WTF::PairHash<int, int>, WTF::PairHashTraits<WTF::UnsignedWithZeroKeyHashTraits<int>, WTF::UnsignedWithZeroKeyHashTraits<int>>> m_idMapping;
-    HashMap<MappedId, IncomingId, WTF::IntHash<MappedId>, WTF::UnsignedWithZeroKeyHashTraits<MappedId>> m_idReverseMapping;
-    MappedId m_primaryId[static_cast<int>(WebPointerProperties::PointerType::LastEntry) + 1];
-    MappedId m_idCount[static_cast<int>(WebPointerProperties::PointerType::LastEntry) + 1];
+    int m_currentId;
+    HashMap<IncomingId, int, WTF::PairHash<int, int>, WTF::PairHashTraits<WTF::UnsignedWithZeroKeyHashTraits<int>, WTF::UnsignedWithZeroKeyHashTraits<int>>> m_idMapping;

[mustaq, 2016/02/06 04:36:31]

Please combine the next two hashmaps into one, this should halve the hash
overhead:
  int -> struct{IncomingId, bool}

Also, to improve readability, make a local typedef for
WTF::UnsignedWithZeroKeyHashTraits<int>.

[Navid Zolghadr, 2016/02/08 15:58:42]

Did you mean m_idReverseMapping and m_lastButtons (-> m_isActive)? Sure. I will
do that.

+    HashMap<int, IncomingId, WTF::IntHash<int>, WTF::UnsignedWithZeroKeyHashTraits<int>> m_idReverseMapping;
+    HashMap<int, int, WTF::IntHash<int>, WTF::UnsignedWithZeroKeyHashTraits<int>> m_lastButtons;

[mustaq, 2016/02/06 04:36:31]

Can you make it <int, bool>? s/m_lastButtons/m_isActive/?

[Navid Zolghadr, 2016/02/08 15:58:42]

Although right now I only use it as bool. But I thought later that I will fire
got/lostpointercapture I need to read this field. But for now I change it to
bool. Maybe later I can get away by passing some extra parameters around.

+    int m_primaryId[static_cast<int>(WebPointerProperties::PointerType::LastEntry) + 1];
+    int m_idCount[static_cast<int>(WebPointerProperties::PointerType::LastEntry) + 1];
+    PointerCapturingMap m_pointerCaptureTarget;
+    PointerCapturingMap m_pendingPointerCaptureTarget;
 };
 
 } // namespace blink
 
 #endif // PointerEventManager_h