Bug 9665

Summary: [Drosera] Conditional breakpoints
Product: WebKit Reporter: Jonathan Hurshman <jhurshman>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Enhancement    
Priority: P3    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Adds conditional breakpoint support
timothy: review-
Updated patch based on irc comments timothy: review+

Jonathan Hurshman
Reported 2006-06-30 04:42:37 PDT
In Venkman, one can create conditional breakpoints, specifying arbitrary JS code which is evaluated in the scope of the breakpoint, and whose return value determines whether the breakpoint stops execution or not. This would be useful to have in Drosera as well.
Attachments
Adds conditional breakpoint support (19.00 KB, patch)
2006-10-06 18:03 PDT, David Smith 		timothy: review-
DetailsFormatted DiffDiff
Updated patch based on irc comments (18.98 KB, patch)
2006-10-06 19:22 PDT, David Smith 		timothy: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Timothy Hatcher
Comment 1 2006-10-05 10:04:54 PDT
Related to bug 11175.
David Smith
Comment 2 2006-10-06 18:03:01 PDT
Created attachment 10957 [details] Adds conditional breakpoint support The breakpoint editor window would likely be a temporary measure until 11175 is implemented, but it'd be easy to switch over once it is.
Timothy Hatcher
Comment 3 2006-10-06 19:18:36 PDT
Comment on attachment 10957 [details] Adds conditional breakpoint support Looking good! A few comments were made on IRC, he is submitting a new patch.
David Smith
Comment 4 2006-10-06 19:22:36 PDT
Created attachment 10960 [details] Updated patch based on irc comments
Mark Rowe (bdash)
Comment 5 2006-10-06 20:35:15 PDT
Landed in r16871.
Timothy Hatcher
Comment 6 2008-05-17 09:55:52 PDT
Closing since Drosera has been replaced by the new Web Inspector debugger. Moving to the New Bugs component so the Drosera component can be closed and removed.
Note You need to log in before you can comment on or make changes to this bug.