Bug 9127 - Invoke capturing event listeners when AT_TARGET
Summary: Invoke capturing event listeners when AT_TARGET
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Anders Carlsson
: 7205 (view as bug list)
Depends on:
Blocks: 7788
  Show dependency treegraph
Reported: 2006-05-26 11:33 PDT by Anders Carlsson
Modified: 2006-06-04 10:07 PDT (History)
3 users (show)

See Also:

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

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]
Comment 3 Anders Carlsson 2006-05-26 11:51:09 PDT
Comment on attachment 8555 [details]

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. ***