Bug 137630

Summary: Add an option to run-webkit-tests to print stderr/stdout from the tool
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Tools / TestsAssignee: David Farler <dfarler>
Status: NEW    
Severity: Normal CC: ap, commit-queue, dbates, ddkilzer, dfarler, glenn, jake.nielsen.webkit
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=168468
Attachments:
Description Flags
Patch bfulgham: review+

Simon Fraser (smfr)
Reported 2014-10-10 17:00:21 PDT
SSIA
Attachments
Patch (3.89 KB, patch)
2014-10-10 18:06 PDT, David Farler 		bfulgham: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Farler
Comment 1 2014-10-10 18:06:47 PDT
Created attachment 239663 [details] Patch
Alexey Proskuryakov
Comment 2 2014-10-10 20:22:27 PDT
--debug-rwt-logging already prints stderr. I'm not quite sure what the use case for the new flag is, but maybe it's best to have it only control stdout printing, and keep stderr for the existing one?
David Farler
Comment 3 2014-10-13 11:40:08 PDT
--debug-rwt-logging is extremely noisy and is for debugging webkitpy code. This option is just to have the results transparently show up without having to go to the result files (I assume that's what smfr is requesting).
Alexey Proskuryakov
Comment 4 2014-10-14 18:50:04 PDT
Makes sense, although it's also fairly easy to just run the tool without run-webkit-tests wrapper to get these. +1 for adding the option.
Brent Fulgham
Comment 5 2014-10-31 10:57:57 PDT
Comment on attachment 239663 [details] Patch This seems like a useful change. r=me.
Note You need to log in before you can comment on or make changes to this bug.