<div style="font-family:arial,helvetica,sans-serif"><font>I created a page with instructions how to use content_shell here: <a href="https://sites.google.com/a/chromium.org/dev/developers/testing/webkit-layout-tests/content-shell">https://sites.google.com/a/chromium.org/dev/developers/testing/webkit-layout-tests/content-shell</a><div>

<br></div><div>At the bottom, I summarized next steps, esp. how we can reuse chromium's test controller for the content shell</div><div><br></div><div>best</div><div>-jochen<br><br><div class="gmail_quote">On Mon, Jun 11, 2012 at 1:53 AM, Darin Fisher <span dir="ltr"><<a href="mailto:darin@chromium.org" target="_blank">darin@chromium.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br><div class="gmail_quote"><div><div class="h5">On Fri, Jun 8, 2012 at 12:51 PM, Jochen Eisinger <span dir="ltr"><<a href="mailto:jochen@chromium.org" target="_blank">jochen@chromium.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br><br><div class="gmail_quote"><div>On Fri, Jun 8, 2012 at 9:41 PM, Ryosuke Niwa <span dir="ltr"><<a href="mailto:rniwa@webkit.org" target="_blank">rniwa@webkit.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div class="gmail_quote"><div>On Fri, Jun 8, 2012 at 12:18 PM, Jochen Eisinger <span dir="ltr"><<a href="mailto:jochen@chromium.org" target="_blank">jochen@chromium.org</a>></span> wrote:</div><div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>I've implemented initial support for running layout tests using chromium's content_shell instead of test_shell as the current DRT implementation does. It's still a very experimental, but might already be interesting for some of you to try.</div>






</blockquote><div><br></div></div><div>This is great! Thanks a lot on working this.</div><div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>1. Make sure your WebKit is at least at r119852 (see <a href="http://trac.webkit.org/wiki/Chromium" target="_blank">http://trac.webkit.org/wiki/Chromium</a> for prerequisites)</div>



<div>2. Apply the attachment from <a href="https://bugs.webkit.org/show_bug.cgi?id=87045" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=87045</a></div><div>3. In Source/WebKit/chromium run gclient sync</div><div>






4. build webkit as usual</div>

<div><br></div><div>E.g. for a debug build on Linux, this should give you out/Debug/content_shell</div><div><br></div><div>You can now run layout tests like this:</div><div><br></div><div>new-run-webkit-tests --chromium --debug --driver-name=content_shell --additional-drt-flag=--dump-render-tree LayoutTests/storage/indexeddb/*</div>








<div><br></div><div>You'll notice that not all tests are passing yet, mainly because not all (or actually, almost none of the) layoutTestController features are implemented yet.</div></blockquote><div><br></div></div>




<div>

Where is layoutTestController implemented? We definitely need to move the implementation of layoutTestController, eventSender, etc... into WebKit repository because we often rename functions, etc... in WebKit. It's unacceptable to require having to modify Chromium code in order to do this refactoring in the future.</div>




</div></blockquote><div><br></div></div><div>It's currently here: <a href="http://code.google.com/searchframe#OAMlx_jo-ck/src/content/shell/layout_test_controller.js&exact_package=chromium" target="_blank">http://code.google.com/searchframe#OAMlx_jo-ck/src/content/shell/layout_test_controller.js&exact_package=chromium</a></div>


<div>

<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div>Per my other thread about sharing IDLs between DumpRenderTree and WebKitTestRunner, I would like to see us sharing IDL with WebKitTestRunner instead of adding yet another binding code.</div>




<div><div><br></div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Another missing feature is producing pixel results. However, I'm currently concentrating on text results, as I think the biggest benefit is the ability to run storage tests on the real storage implementation.</div>






</blockquote><div><br></div></div><div>That sounds great to me but we definitely need to support pixel results eventually. I'm more than happy to help you on that but that requires the codebase to be moved into WebKit repository.</div>




</div></blockquote><div><br></div></div><div>Here's the basic problem: content_shell depends on content, so moving this on the webkit repository would mean that webkit depended on content.</div><div><br></div><div>Another solution would be to formalize the test shell API our current layout test controller in webkit uses (Tools/DumpRenderTree/chromium), and add a method to chromium's webkit support library that returns a webview that supports all the hooks required. The webview could then either be implemented by test_shell or by content_shell</div>




<div><br></div></div></blockquote><div><br></div></div></div><div>^^^ This second solution seems like it will work well.  It will enable the guts of layoutTestController to remain in the WebKit repository.  This is just a variation of exactly what we do today.  You only need to move creation of WebView to Chromium so that we can eliminate WebViewHost.cpp (and other "simple" application shell bits).</div>

<span class="HOEnZb"><font color="#888888">
<div><br></div><div>-Darin</div></font></span><div class="im"><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div>

</div><div>best</div><span><font color="#888888"><div>
-jochen</div></font></span></div>
<br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></blockquote></div></div><br>
</blockquote></div><br></div></font></div>