[Webkit-unassigned] [Bug 90675] Web Inspector: implement testing harness for pure protocol tests.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Jul 13 00:37:25 PDT 2012


https://bugs.webkit.org/show_bug.cgi?id=90675





--- Comment #9 from Pavel Feldman <pfeldman at chromium.org>  2012-07-13 00:37:24 PST ---
Capturing the IRC discussion here:

So the (ideal) plan would be:

1) Implement window.internals.openInspectorTestFrontend

This method will create a Page, install the InspectorFrontendClientLocal-based harness on it (a-la Qt or GTK port). This should require almost no code.

2) Complete the refactoring stated in https://bugs.webkit.org/show_bug.cgi?id=91196. After that, we'll be able to call connectFrontend with the transport from our new test harness.

3) Figure out a way to evaluate InspectorBackend.js and InspectorBackendCommands.js in the context of the new front-end Page. We can use .js -> .h method that is currently used for InjectedScriptSource.js. It creates a header from the .js file with the string constant that can be used from native code.

4) Protocol layout tests will include the modified / stripped version of the inspector-test.js. This version will use new

window.internal.openDummyInspectorFrontend() instead of testRunner.showWebInspector() and same for closeWebInspector.

5) testRunner.evaluateInWebInspector should continue functioning.

6) New tests will live under LayoutTests/inspector-enabled. That's the folder that allows inspector tests, but does not open inspector automatically. DRTs should be modified to do window.internal.closeDummyInspectorFrontend after tests from that folder. 

7) After that, some of the tests (that only talk to the backend), could be moved from LayoutTests/inspector to LayoutTests/inspector-enabled and start using new harness.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list