Bug 3496

Summary: Add gdb-safari script to launch Safari under gdb
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: Tools / TestsAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Enhancement    
Priority: P2    
Version: 412   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Fix
none
New fix
none
gdb-safari script none

Anders Carlsson
Reported 2005-06-12 23:17:33 PDT
It would be nice if run-safari --debug would launch Safari under gdb. I noticed that there already is a -- debug option but that it does nothing, so I made a patch that launches it under gdb.
Attachments
Fix (1.45 KB, patch)
2005-06-12 23:18 PDT, Anders Carlsson 		no flags
DetailsFormatted DiffDiff
New fix (1.66 KB, patch)
2005-06-12 23:47 PDT, Anders Carlsson 		no flags
DetailsFormatted DiffDiff
gdb-safari script (2.30 KB, application/octet-stream)
2005-06-13 23:39 PDT, Anders Carlsson 		no flags
Details
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2005-06-12 23:18:24 PDT
Created attachment 2289 [details] Fix
Anders Carlsson
Comment 2 2005-06-12 23:47:31 PDT
Created attachment 2291 [details] New fix Whoops, looks like run-safari has changed. Here's a new version that actually works
Darin Adler
Comment 3 2005-06-13 08:43:39 PDT
The old --debug was really a bad name for --development. I think that it would be good to be able to launch under gdb, but I'd prefer a script named gdb-safari to do that. I know people can make aliases, but I want the default names to be good.
Anders Carlsson
Comment 4 2005-06-13 23:39:23 PDT
Created attachment 2314 [details] gdb-safari script Yeah, that sounds like a better idea. Here's a gdb-safari script.
Darin Adler
Comment 5 2005-06-14 12:01:17 PDT
I tweaked the script a little bit and landed it.
Chris Petersen
Comment 6 2005-06-18 23:13:03 PDT
Reporter, please mark this bug as Verified if this issue has been fixed in the latest TOT Webkit.
Joost de Valk (AlthA)
Comment 7 2005-07-03 08:00:48 PDT
Reporter, please mark bug as verified :)
Note You need to log in before you can comment on or make changes to this bug.