Bug 148181

Summary:

Web Inspector: add TestHarness option to tee all commands to system console

Product:

WebKit

Reporter:

BJ Burg <bburg>

Component:

Web Inspector

Assignee:

BJ Burg <bburg>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bburg, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

147093

Attachments:

Description	Flags
Patch	joepeck: review+

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>