Bug 137662

Summary:

[iOS] LayoutTestRelay: Detect broken pipe when webkitpy cancels a run

Product:

WebKit

Reporter:

David Farler <dfarler>

Component:

Tools / Tests

Assignee:

David Farler <dfarler>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, ddkilzer, simon.fraser

Priority:

Version:

528+ (Nightly build)

Hardware:

iPhone / iPad

OS:

All

Attachments:

Description	Flags
Patch	darin: review+

David Farler

Reported 2014-10-13 11:28:25 PDT

Rarely, with proper timing, the LayoutTestRelay's simulator app can crash while the didReceiveStderrData and didReceiveStdoutData NSFileHandle delegate methods are running, but the pipe will be broken, so LayoutTestRelay will crash. We should catch NSFileHandleOperationException and call didCrashWithMessage, same as on the write side. We have broken pipe coverage for: webkitpy -> [stdin] LayoutTestRelay -> [stdin] DRT and need coverage for: webkitpy [stdout result] <- LayoutTestRelay [NSFileHandle] <- [stdout] DRT webkitpy [stderr result] <- LayoutTestRelay [NSFileHandle] <- [stderr] DRT

Attachments
Patch (2.44 KB, patch) 2014-10-13 11:45 PDT, David Farler	darin: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

David Farler

Comment 1 2014-10-13 11:28:38 PDT

<rdar://problem/18635888>

David Farler

Comment 2 2014-10-13 11:38:45 PDT

Actually wait, this is on the webkitpy side, so I think this might actually be the case where we timed out, or a run was cancelled, and webkitpy stopped reading our stdout/stderr. I think the proper thing to do is just exit(1) here.

David Farler

Comment 3 2014-10-13 11:45:03 PDT

Created attachment 239736 [details] Patch

WebKit Commit Bot

Comment 4 2014-10-13 11:46:35 PDT

Attachment 239736 [details] did not pass style-queue: ERROR: Tools/LayoutTestRelay/LayoutTestRelay/LTRelayController.m:110: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/LayoutTestRelay/LayoutTestRelay/LTRelayController.m:110: Extra space before ( in function call [whitespace/parens] [4] ERROR: Tools/LayoutTestRelay/LayoutTestRelay/LTRelayController.m:122: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Tools/LayoutTestRelay/LayoutTestRelay/LTRelayController.m:122: Extra space before ( in function call [whitespace/parens] [4] Total errors found: 4 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.

Darin Adler

Comment 5 2014-10-15 09:40:55 PDT

Comment on attachment 239736 [details] Patch I suggest naming the exception “exception” rather than “e”.

David Farler

Comment 6 2014-10-15 10:01:16 PDT

Committed r174728: <http://trac.webkit.org/changeset/174728>

David Farler

Comment 7 2014-10-15 10:01:33 PDT

Thanks Darin -

Note You need to log in before you can comment on or make changes to this bug.