216237 - Stopping event propagation should not cancel system event group handlers

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Brian Ryner (not reading)]

If you cancel an event by calling event.stopPropagation(),
event.preventBubble(), event.preventCapture(), or event.cancelBubble = true,
then we incorrectly apply that to the system event group as well.  This violates
section 1.2.2.6 of the DOM3 Events spec, which says that stopping propagation in
one event group cannot affect propagation in other event groups.  Besides that,
it leads to some nasty problems, which my testcase will demonstrate.

Comment 1

[Brian Ryner (not reading)]

Attachment: testcase

You should be able to type in the text field on this page, but you can't.

Comment 2

[Brian Ryner (not reading)]

Attachment: patch

Clear the STOP_DISPATCH flag before beginning the second event pass.

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 129837 [details] [diff] [review]
patch

sr=jst

Comment 4

[Brian Ryner (not reading)]

Comment on attachment 129837 [details] [diff] [review]
patch

Can I get this in for 1.5b?  It's a pretty safe fix.

Comment 5

[Mike Kaply [:mkaply]]

Comment on attachment 129837 [details] [diff] [review]
patch

a=mkaply for trunk and branch

Thanks for finding this

Comment 6

[Brian Ryner (not reading)]

Checked in on trunk and 1.4 branch.