Consider isPrimary and pointerType when preventing compat mouse

third_party/WebKit/Source/core/input/PointerEventManager.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 PointerEventManager_h
 #define PointerEventManager_h
 
 #include "core/CoreExport.h"
 #include "core/events/PointerEvent.h"
 #include "core/events/PointerEventFactory.h"
@@ skipping 114 matching lines @@
     void removePointer(const PassRefPtrWillBeRawPtr<PointerEvent>);
     WebInputEventResult dispatchPointerEvent(
         PassRefPtrWillBeRawPtr<EventTarget>,
         PassRefPtrWillBeRawPtr<PointerEvent>,
         bool checkForListener = false);
     void releasePointerCapture(int);
 
     // Prevents firing mousedown, mousemove & mouseup in-between a canceled pointerdown and next pointerup/pointercancel.
     // See "PREVENT MOUSE EVENT flag" in the spec:
     //   https://w3c.github.io/pointerevents/#compatibility-mapping-with-mouse-events
-    bool m_preventMouseEventForPointerTypeMouse;
+    bool m_preventMouseEventForPointerType[static_cast<int>(WebPointerProperties::PointerType::LastEntry) + 1];

[dtapuska, 2016/04/05 19:21:25]

Personally I'd cast to a size_t

[Navid Zolghadr, 2016/04/05 20:17:25]

Done.

 
     // Note that this map keeps track of node under pointer with id=1 as well
     // which might be different than m_nodeUnderMouse in EventHandler. That one
     // keeps track of any compatibility mouse event positions but this map for
     // the pointer with id=1 is only taking care of true mouse related events.
     WillBeHeapHashMap<int, EventTargetAttributes> m_nodeUnderPointer;
 
     PointerCapturingMap m_pointerCaptureTarget;
     PointerCapturingMap m_pendingPointerCaptureTarget;
     PointerEventFactory m_pointerEventFactory;
 };
 
 } // namespace blink
 
 #endif // PointerEventManager_h