Bug 148181 - Web Inspector: add TestHarness option to tee all commands to system console
Summary: Web Inspector: add TestHarness option to tee all commands to system console
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: BJ Burg
URL:
Keywords: InRadar
Depends on:
Blocks: InspectorTest
  Show dependency tree / graph
 
Reported: 2015-08-19 12:10 PDT by BJ Burg
Modified: 2015-08-20 09:44 PDT (History)
7 users (show)

See Also:

Attachments
Patch (12.26 KB, patch)
2015-08-19 15:36 PDT, BJ Burg 		joepeck: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description BJ Burg 2015-08-19 12:10:46 PDT 
It should be possible to dump test output to stderr with a single line at the top of the file. This will be useful in debugging test timeouts.
Comment 1 Radar WebKit Bug Importer 2015-08-19 12:11:41 PDT 
<rdar://problem/22347806>
Comment 2 BJ Burg 2015-08-19 15:36:52 PDT 
Created attachment 259417 [details]
Patch
Comment 3 Joseph Pecoraro 2015-08-19 16:12:08 PDT 
Comment on attachment 259417 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=259417&action=review

r=me

> Source/WebCore/inspector/InspectorFrontendHost.cpp:311
> -    WTFLogAlways("InspectorTest: %s", message.utf8().data());
> +    WTFLogAlways("%s", message.utf8().data());

It is often nice to have something to grep for, perhaps here you are thinking just use the process name / pid in output?
Comment 4 BJ Burg 2015-08-20 09:44:07 PDT 
Committed r188687: <http://trac.webkit.org/changeset/188687>