Bug 127125

Summary: Web Inspector: add more context to Object.addEventListener assertions
Product: WebKit Reporter: Blaze Burg <bburg>
Component: WebCore Misc.Assignee: Blaze Burg <bburg>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, joepeck, timothy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch none

Blaze Burg
Reported 2014-01-16 11:29:45 PST
I seem to find myself adding this back in frequently when debugging events.. might as well submit it.
Attachments
patch (1.78 KB, patch)
2014-01-16 11:37 PST, Blaze Burg 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Blaze Burg
Comment 1 2014-01-16 11:37:11 PST
Created attachment 221396 [details] patch
Joseph Pecoraro
Comment 2 2014-01-16 12:52:54 PST
Comment on attachment 221396 [details] patch r=me Maybe you could even do console.assert(condition, "reason", "(", arguments, ")")
Blaze Burg
Comment 3 2014-01-16 12:55:16 PST
I don't think console will understand how to display arguments.
WebKit Commit Bot
Comment 4 2014-01-16 13:48:31 PST
Comment on attachment 221396 [details] patch Clearing flags on attachment: 221396 Committed r162149: <http://trac.webkit.org/changeset/162149>
WebKit Commit Bot
Comment 5 2014-01-16 13:48:33 PST
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.