Bug 16446

Summary: Edits to expand on Windows debug environment
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: WebKit WebsiteAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: mrowe
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch to expand debugging instructions for Windows.
none
screenshot for website changes
none
Screenshot for setting "/console" flag in VS2005
none
Website update including new VS 2005 screenshots. mjs: review+

Brent Fulgham
Reported 2007-12-14 22:52:34 PST
The instructions for debugging under Windows omit some useful details. This patch adds a couple of screenshots and some explanatory text.
Attachments
Patch to expand debugging instructions for Windows. (1.71 KB, patch)
2007-12-14 22:53 PST, Brent Fulgham 		no flags
DetailsFormatted DiffDiff
screenshot for website changes (7.99 KB, image/jpeg)
2007-12-14 22:54 PST, Brent Fulgham 		no flags
Details
Screenshot for setting "/console" flag in VS2005 (44.20 KB, image/jpeg)
2007-12-14 22:55 PST, Brent Fulgham 		no flags
Details
Website update including new VS 2005 screenshots. (72.94 KB, patch)
2007-12-16 14:35 PST, Brent Fulgham 		mjs: review+
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
Brent Fulgham
Comment 1 2007-12-14 22:53:47 PST
Created attachment 17903 [details] Patch to expand debugging instructions for Windows.
Brent Fulgham
Comment 2 2007-12-14 22:54:28 PST
Created attachment 17904 [details] screenshot for website changes
Brent Fulgham
Comment 3 2007-12-14 22:55:08 PST
Created attachment 17905 [details] Screenshot for setting "/console" flag in VS2005
Mark Rowe (bdash)
Comment 4 2007-12-14 23:16:07 PST
Brent, can you "svn add" the images and use "svn-create-patch" to create the patch? That will include the screenshots in the patch and make it a little easier to land.
Adam Roben (:aroben)
Comment 5 2007-12-15 09:37:45 PST
Comment on attachment 17903 [details] Patch to expand debugging instructions for Windows. These instructions look great! +<li><p>If you want to see the helpful output debug messages, you should edit the debug settings for the Safari solution (i.e., the pseudo-project created when the debugger attached to Safari):</p> +<img src="debug_vs2005.jpg"> +<p>You should consider including the command-line switch '<span class="code">/console</span>'. This causes Safari to run with an open DOS shell in which output messages (created with the <span class="code">LOG</span> macro).</p> +<img src="console_vs2005.jpg"></li> I wonder if we should just make debug-safari do this by default? It would require changing FindSafari.
Brent Fulgham
Comment 6 2007-12-16 14:35:46 PST
Created attachment 17940 [details] Website update including new VS 2005 screenshots. Updated per comments in this bug report.
Brent Fulgham
Comment 7 2007-12-16 14:36:07 PST
Comment on attachment 17940 [details] Website update including new VS 2005 screenshots. Updated per notes on bug.
Maciej Stachowiak
Comment 8 2007-12-16 14:36:49 PST
Comment on attachment 17940 [details] Website update including new VS 2005 screenshots. r=me
Mark Rowe (bdash)
Comment 9 2007-12-16 17:21:17 PST
Landed in r28783.
Note You need to log in before you can comment on or make changes to this bug.