Prevent sending click event for non primary button

third_party/WebKit/Source/core/events/MouseEvent.cpp

Index: third_party/WebKit/Source/core/events/MouseEvent.cpp
diff --git a/third_party/WebKit/Source/core/events/MouseEvent.cpp b/third_party/WebKit/Source/core/events/MouseEvent.cpp
index 943a33e71c2d3d2797810a8027c2d55900541b6d..c34468cf0de173777e20df983294f52ef70d57e5 100644
--- a/third_party/WebKit/Source/core/events/MouseEvent.cpp
+++ b/third_party/WebKit/Source/core/events/MouseEvent.cpp
@@ -294,6 +294,10 @@ DispatchEventResult MouseEventDispatchMediator::dispatchEvent(EventDispatcher& d
     if (mouseEvent.type() != EventTypeNames::click || mouseEvent.detail() != 2)
         return dispatchResult;
 
+    // Do not send dblclick event for middle click
+    if (mouseEvent.button() == MouseButton::MiddleButton)

[dtapuska, 2016/04/22 18:32:30]

I don't think this is what the spec indicates. It indicates click should only be
sent for the primary button; ie; what about right click?

[Navid Zolghadr, 2016/04/22 19:09:58]

Right. So much focusing on middlebutton that I forgot that here. I used
!=leftbutton in other places though.

+        return dispatchResult;
+
     // Special case: If it's a double click event, we also send the dblclick event. This is not part
     // of the DOM specs, but is used for compatibility with the ondblclick="" attribute. This is treated
     // as a separate event in other DOM-compliant browsers like Firefox, and so we do the same.