Bug 77331

Summary: Web Inspector: should be possible to step through all event listeners when event listener breakpoint is hit
Product: WebKit Reporter: Yury Semikhatsky <yurys>
Component: Web Inspector (Deprecated)Assignee: Yury Semikhatsky <yurys>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch pfeldman: review+

Description Yury Semikhatsky 2012-01-30 04:52:42 PST 
Now debugger will only pause in the first listener and stepping out will resume execution.
Comment 1 Yury Semikhatsky 2012-01-30 10:23:38 PST 
Created attachment 124569 [details]
Patch
Comment 2 Yury Semikhatsky 2012-01-30 10:25:54 PST 
(In reply to comment #1)
> Created an attachment (id=124569) [details]
> Patch

I considered moving new instrumentation calls into V8AbstractEventListener/JSEventListener classes in order to have instrumentation only for JS listeners but decided to leave it in EventTarget as it has a single implementation for both engines.
Comment 3 Yury Semikhatsky 2012-01-30 22:54:03 PST 
Created attachment 124676 [details]
Patch
Comment 4 Yury Semikhatsky 2012-01-30 22:57:12 PST 
(In reply to comment #3)
> Created an attachment (id=124676) [details]
> Patch

Same change with a test.
Comment 5 Yury Semikhatsky 2012-01-30 23:37:55 PST 
Committed r106337: <http://trac.webkit.org/changeset/106337>