RESOLVED FIXED 52927
nrwt: handle broken pipe errors from DRT more cleanly 
https://bugs.webkit.org/show_bug.cgi?id=52927
Summary nrwt: handle broken pipe errors from DRT more cleanly
Dirk Pranke
Reported 2011-01-21 15:01:07 PST
nrwt: handle broken pipe errors from DRT more cleanly
Attachments
Patch (1.32 KB, patch)
2011-01-21 15:02 PST, Dirk Pranke 		no flags
DetailsFormatted DiffDiff
add unit test for broken pipes (4.56 KB, patch)
2011-02-02 17:49 PST, Dirk Pranke 		tony: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dirk Pranke
Comment 1 2011-01-21 15:02:09 PST
Created attachment 79799 [details] Patch
Eric Seidel (no email)
Comment 2 2011-01-22 15:10:01 PST
Comment on attachment 79799 [details] Patch Looks reasonable. Seems we should have a test though.
Dirk Pranke
Comment 3 2011-01-23 10:22:11 PST
I thought about this a fair amount before uploading the patch, and it's not clear to me how to usefully test this. To do so either requires spawning off a process that crashes at exactly the right point (since most of the time I'll pick up the crash during the read loop) or mocking out several layers of calls, which seems overly contrived and kind of pointless. I'm open to suggestions, though; any ideas on how you would test it?
Eric Seidel (no email)
Comment 4 2011-01-23 11:09:22 PST
Comment on attachment 79799 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=79799&action=review > Tools/Scripts/webkitpy/layout_tests/port/server_process.py:119 > + self._proc.stdin.write(input) Oh, I had assumed one would just mock the multiple layers. I don't know how much other mocking has already been done for testing this class.
Dirk Pranke
Comment 5 2011-02-02 17:49:28 PST
Created attachment 81015 [details] add unit test for broken pipes
Dirk Pranke
Comment 6 2011-02-03 20:07:20 PST
Committed r77592: <http://trac.webkit.org/changeset/77592>
Note You need to log in before you can comment on or make changes to this bug.