Issue metadata
Sign in to add a comment
|
Double-click events not fired on SVG <use> elements
Reported by
ken...@gmail.com,
Nov 4 2016
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.87 Safari/537.36 Steps to reproduce the problem: With the attached file: 1.Double-click on the text "double click test" - JS event is fired, correctly 2. Double-click on the black circle - JS event is not fired. What is the expected behavior? The event should be fired when double clicking the <use> element What went wrong? The event was not fired. Did this work before? Yes 53 Does this work in other browsers? Yes Chrome version: 54.0.2840.87 Channel: stable OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version: Shockwave Flash 23.0 r0
,
Nov 4 2016
Let's verify the breaking change.
,
Nov 4 2016
It is most likely https://codereview.chromium.org/2186823002. In eventPathShouldBeEmptyFor(...) the single click event is composed() but the dblclick follow up is not.
,
Nov 4 2016
And the reason it's not composed is because the event created in MouseEventDispatchMediator::dispatchEvent for the double click uses default MouseEvent constructor which does not invoke any other constructor to set the composed flag. That seems like a bug because I don't see why click would be composed() but not doubleclick, but I don't know enough about events to know whether it's important. That means this is really an Events bug, as the issue is construction of the double click event.
,
Nov 7 2016
Sounds a bug. Let me take a look.
,
Nov 9 2016
,
Nov 10 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/0eec66d250f63a4802bf17b0d0b84a8ddeda91f6 commit 0eec66d250f63a4802bf17b0d0b84a8ddeda91f6 Author: hayato <hayato@chromium.org> Date: Thu Nov 10 03:49:41 2016 Make UA dblclick events composed events This is a bug fix. BUG= 662401 Review-Url: https://codereview.chromium.org/2490883002 Cr-Commit-Position: refs/heads/master@{#431172} [modify] https://crrev.com/0eec66d250f63a4802bf17b0d0b84a8ddeda91f6/third_party/WebKit/LayoutTests/shadow-dom/event-composed-ua.html [modify] https://crrev.com/0eec66d250f63a4802bf17b0d0b84a8ddeda91f6/third_party/WebKit/Source/core/events/Event.h [modify] https://crrev.com/0eec66d250f63a4802bf17b0d0b84a8ddeda91f6/third_party/WebKit/Source/core/events/MouseEvent.cpp
,
Nov 10 2016
,
Mar 15 2017
Remove Blink>DOM>Events |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by f...@opera.com
, Nov 4 2016Labels: -OS-Windows
Status: Available (was: Unconfirmed)