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

Unified Diff: third_party/WebKit/Source/core/events/PointerEventFactory.h

Issue 1670073004: Send node transition events for touch events (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Applying the comments Created 4 years, 10 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/events/PointerEventFactory.h
diff --git a/third_party/WebKit/Source/core/events/PointerEventFactory.h b/third_party/WebKit/Source/core/events/PointerEventFactory.h
new file mode 100644
index 0000000000000000000000000000000000000000..36957565eab9e29ea26a6129201bce412256927a
--- /dev/null
+++ b/third_party/WebKit/Source/core/events/PointerEventFactory.h
@@ -0,0 +1,80 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
Ian Vollick 2016/02/12 15:01:17 2016
Navid Zolghadr 2016/02/12 16:30:47 Done.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef PointerEventFactory_h
+#define PointerEventFactory_h
+
+#include "core/CoreExport.h"
+#include "core/events/PointerEvent.h"
+#include "public/platform/WebPointerProperties.h"
+#include "wtf/Allocator.h"
+#include "wtf/HashMap.h"
+
+namespace blink {
+
+/**
+ Helper class for tracking the pointer ids for each type of PointerEvents.
+ Using id re-mapping at this layer, this class makes sure that regardless of
+ the domain of the id (i.e. in touch or pen) the corresponding pointer event
+ will have a unique id amongst all pointer events as per pointer events'
Ian Vollick 2016/02/12 15:01:17 nit: I'd use // to be consistent with the license
Navid Zolghadr 2016/02/12 16:30:47 Done.
+ specification. This class intended to behave the same as existing browsers as
+ much as possible for compatibility reasons. Particularly it behaves very
+ similar to MS Edge to have pointer event ids generated by mouse always equal 1
+ and those that are generated by touch and pen will have increasing ids from 2.
+*/
+class CORE_EXPORT PointerEventFactory {
+ DISALLOW_NEW();
+public:
+
+ PointerEventFactory();
+ ~PointerEventFactory();
+
+ PassRefPtrWillBeRawPtr<PointerEvent> create(const AtomicString& type,
+ const PlatformMouseEvent&, PassRefPtrWillBeRawPtr<Node> relatedTarget,
+ PassRefPtrWillBeRawPtr<AbstractView>);
+
+ PassRefPtrWillBeRawPtr<PointerEvent> create(const AtomicString& type,
+ const PlatformTouchPoint&, PlatformEvent::Modifiers,
+ const double width, const double height,
+ const double clientX, const double clientY);
+
+ PassRefPtrWillBeRawPtr<PointerEvent> createPointerCancel(
+ const PlatformTouchPoint&);
+
+ PassRefPtrWillBeRawPtr<PointerEvent> create(
+ PassRefPtrWillBeRawPtr<PointerEvent>,
+ const AtomicString& type,
+ PassRefPtrWillBeRawPtr<EventTarget>);
+
+ // Clear all the existing ids.
+ void clear();
+
+ /*
+ * 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>);
+
+private:
+ typedef std::pair<int, int> IncomingId;
+ typedef WTF::UnsignedWithZeroKeyHashTraits<int> UnsignedHash;
+
+ int add(const IncomingId);
+ bool isPrimary(const int) const;
+ void setIdAndType(PointerEventInit &, const WebPointerProperties &);
+
+ static const int s_invalidId;
+ static const int s_mouseId;
+
+ int m_currentId;
+ HashMap<IncomingId, int, WTF::PairHash<int, int>, WTF::PairHashTraits<UnsignedHash, UnsignedHash>> m_idMapping;
+ HashMap<int, IncomingId, WTF::IntHash<int>, UnsignedHash> m_idReverseMapping;
+ int m_primaryId[static_cast<int>(WebPointerProperties::PointerType::LastEntry) + 1];
+ int m_idCount[static_cast<int>(WebPointerProperties::PointerType::LastEntry) + 1];
+};
+
+} // namespace blink
+
+#endif // PointerEventFactory_h

Powered by Google App Engine
This is Rietveld 408576698