Bug 9127

Summary: Invoke capturing event listeners when AT_TARGET
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: DOMAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: annevk, kjb525, mjs
Priority: P2    
Version: 420+   
Hardware: Macintosh   
OS: OS X 10.4   
Bug Depends on:    
Bug Blocks: 7788    
Attachments:
Description Flags
Patch
andersca: review-
Better approach ggaren: review+

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