RESOLVED FIXED 31030
Web Inspector: Timeline instrumentation for Node::dispatchGenericEvent missed a case. 
https://bugs.webkit.org/show_bug.cgi?id=31030
Summary Web Inspector: Timeline instrumentation for Node::dispatchGenericEvent missed...
Kelly Norton
Reported Monday, November 2, 2009 9:35:17 PM UTC
This one is my bad. When I filtered out events with no listeners, I forgot to add support for DOMWindow dispatch. This only effects DOM events that are delivered to window (like 'click'). So we current miss this case: window.addEventListener('click', function(e) { }, false); Patch is ready, I'm about to upload.
Attachments
Proposed fix. (2.63 KB, patch)
2009-11-02 13:42 PST, Kelly Norton 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kelly Norton
Comment 1 Monday, November 2, 2009 9:42:50 PM UTC
Created attachment 42340 [details] Proposed fix.
WebKit Commit Bot
Comment 2 Monday, November 2, 2009 10:02:40 PM UTC
Comment on attachment 42340 [details] Proposed fix. Clearing flags on attachment: 42340 Committed r50428: <http://trac.webkit.org/changeset/50428>
WebKit Commit Bot
Comment 3 Monday, November 2, 2009 10:02:44 PM UTC
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.