Bug 26549 – Web Inspector needs a testing infrastructure

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Kevin McCullough]

* SUMMARY
We have many involved and highly promoted features in the Web Inspector but with almost no way to test most of them.  (e.g. the JS debugger and profiler)

We should build a testing infrastructure similar to DRT that allows the Inspector to be tested with simple JS and HTML files so that developers can create tests like the LayoutTests, and make them easy to run by developers and the build bots.

[Mark Rowe (bdash)]

<rdar://problem/6988926>

[Pavel Feldman]

Created attachment 40370 [details]
testing harness

This patch contains two test cases and the harness implementation for the Mac. I'd like to get your feedback on it before I extract WebInspectorPrivate.h and jump into implementing it for other platforms.

[Timothy Hatcher]

Comment on attachment 40370 [details]
testing harness

Some tabs in evaluate-in-frontend.js.

> +	var output = document.getElementById("output");
> +    output.innerHTML += text + "<BR>";

Stray space.

> +    void testEvaluateInFrontend(long callId, const String& script);

These functions with "testEvaluate" read funny. I think "evaluateTest" would read better. Since I read it as "evaluate this test", not "test the evaluate".

> +    Vector<pair<long, String> > m_pendingTestEvaluateInFrontendCommands;

Maybe m_pendingEvaluateTestCommands.

[Pavel Feldman]

Created attachment 40470 [details]
testing harness #2

[Pavel Feldman]

Created attachment 40472 [details]
testing harness #2 with waitToDumpWatchdogInterval rolled back to original value

Note that this patch requires additional testing on non-Mac platforms prior to landing.

[Pavel Feldman]

I am splitting this patch into several ones:
https://bugs.webkit.org/show_bug.cgi?id=30008
https://bugs.webkit.org/show_bug.cgi?id=30009
https://bugs.webkit.org/show_bug.cgi?id=30010
https://bugs.webkit.org/show_bug.cgi?id=30014

[Timothy Hatcher]

Comment on attachment 40472 [details]
testing harness #2 with waitToDumpWatchdogInterval rolled back to original value

Clearing the review flag since this patch has been split.