9127 – Invoke capturing event listeners when AT_TARGET

Bug 9127 - Invoke capturing event listeners when AT_TARGET

Summary: Invoke capturing event listeners when AT_TARGET

Status:	RESOLVED FIXED

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	DOM (show other bugs)
Version:	420+
Hardware:	Mac OS X 10.4

Importance:	P2 Normal
Assignee:	Anders Carlsson

URL:
Keywords:

Duplicates (1):	7205 (view as bug list)
Depends on:
Blocks:	7788
	Show dependency tree / graph

Reported:	2006-05-26 11:33 PDT by Anders Carlsson
Modified:	2006-06-04 10:07 PDT (History)
CC List:	3 users (show)

See Also:

Attachments
Patch (4.81 KB, patch) 2006-05-26 11:46 PDT, Anders Carlsson	andersca: review-	Details \| Formatted Diff \| Diff
Better approach (4.79 KB, patch) 2006-05-26 12:00 PDT, Anders Carlsson	ggaren: review+	Details \| Formatted Diff \| Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Anders Carlsson 2006-05-26 11:33:23 PDT

This is contrary to the DOM 2 spec which says that

"A capturing EventListener will not be triggered by events dispatched directly to the EventTarget upon which it is registered."

However, Mozilla invokes capturing event listeners when AT_TARGET and live.com (at least) depends on that so we should do it too

Comment 1 Dave Hyatt 2006-05-26 11:42:54 PDT

We used to work this way actually, so this is actually a regression from Safari 1.2.  I added code back in 1.2 to make sure we did capturing, then a capture at target, then a bubble at target, then bubbling.  At some point the capture at the target must have been removed.

Comment 2 Anders Carlsson 2006-05-26 11:46:14 PDT

Created attachment 8555 [details]
Patch

Comment 3 Anders Carlsson 2006-05-26 11:51:09 PDT

Comment on attachment 8555 [details]
Patch

I'll dig up the way hyatt implemented it and use that

Comment 4 Anders Carlsson 2006-05-26 12:00:21 PDT

Created attachment 8557 [details]
Better approach

Comment 5 Geoffrey Garen 2006-05-26 13:09:12 PDT

Comment on attachment 8557 [details]
Better approach

I'm marking this r+ after talking with Anders and Maciej. We want to emulate the Firefox to support live.com.

The Firefox bugzilla about this is: https://bugzilla.mozilla.org/show_bug.cgi?id=235441.

Comment 6 Anne van Kesteren 2006-05-27 14:49:50 PDT

Maciej, weren't you guys planning to not do this? This seems wrong.

Comment 7 Darin Adler 2006-06-04 10:07:38 PDT

*** Bug 7205 has been marked as a duplicate of this bug. ***