Summary: | Fix a bug that mousedown without mouseup in a frame disturbs click event in another frame | ||
---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
Component: | UI Events | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | ahmad.saleem792, ap, benjamin, enrica, tkent, webkit-bug-importer |
Priority: | P2 | Keywords: | BlinkMergeCandidate, InRadar |
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Description
Ryosuke Niwa
2013-07-30 20:29:22 PDT
This is still something can be implemented: https://github.com/WebKit/WebKit/blob/2ef8bf040a9db8c0af836f5a92916e087e9b9e16/Source/WebCore/page/EventHandler.cpp#L1711 but I don't know alternative names of: m_eventHandlerWillResetCapturingMouseEventsNode -> can't figure out m_capturingMouseEventsNode -> could be m_capturingMouseEventsElement Plus I was not able to test the attached testcase because it requires eventSender, which might be only possible in EWS. Thanks! (In reply to Ahmad Saleem from comment #1) > This is still something can be implemented: > > https://github.com/WebKit/WebKit/blob/ > 2ef8bf040a9db8c0af836f5a92916e087e9b9e16/Source/WebCore/page/EventHandler. > cpp#L1711 > > but I don't know alternative names of: > > m_eventHandlerWillResetCapturingMouseEventsNode -> can't figure out > > m_capturingMouseEventsNode -> could be m_capturingMouseEventsElement > > Plus I was not able to test the attached testcase because it requires > eventSender, which might be only possible in EWS. Thanks! We also have "clearLatchedState", we can call it as well like: if (xx) return clearLatchedState; xx -> m_eventHandlerWillResetCapturingMouseEventsNode (still not clear alternative in Webkit). Committed 258055@main (c2dd7ba98908): <https://commits.webkit.org/258055@main> Reviewed commits have been landed. Closing PR #7811 and removing active labels. |