<br><br><div class="gmail_quote">On Tue, Apr 13, 2010 at 11:39 PM, David Levin <span dir="ltr">&lt;<a href="mailto:levin@google.com">levin@google.com</a>&gt;</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 class="im">On Tue, Apr 13, 2010 at 11:35 PM, Adam Barth <span dir="ltr">&lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Ah, sorry, I meant to the point where the library was integrated with<br>
the various build systems, etc.  Maybe that&#39;s already possible today.<br>
I haven&#39;t investigated it in detail.  It probably also makes sense to<br>
talk about correctness between steps (1) and (2), once we have the<br>
tests and their results to compare across browsers.<br>
<font color="#888888"><br>
Adam<br>
</font><div><div></div><div><br>
<br>
On Tue, Apr 13, 2010 at 11:28 PM, Darin Fisher &lt;<a href="mailto:darin@chromium.org" target="_blank">darin@chromium.org</a>&gt; wrote:<br>
&gt; #4 is already basically done.  See USE(GOOGLEURL).<br>
&gt; -Darin<br>
&gt;<br>
&gt;<br>
&gt; On Tue, Apr 13, 2010 at 11:17 PM, Adam Barth &lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; I think Maciej took notes, but my recollection is as follows:<br>
&gt;&gt;<br>
&gt;&gt; 1) Convert as many of the unit tests to LayoutTests as possible.<br>
&gt;&gt; 2) Land GURL in <a href="http://svn.webkit.org" target="_blank">svn.webkit.org</a> as is.<br></div></div></blockquote><div><br></div></div><div>I think there was: </div><div>   2.5 Change the name from GoogleURL to something else.</div>

<div> </div><div>(WebKitUrl?)</div><div><div></div><div class="h5"><div><br></div></div></div></div></blockquote><div><br></div><div>WebKitUrl suggests that it lives more at the higher levels of the code base (with the other WebKit* stuff).  How about URLCore living at the same level as WebCore and JavaScriptCore?</div>
<div><br></div><div>-Darin</div><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 class="h5"><div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div>
&gt;&gt; 3) Convert GURL to WebKit style. </div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>
&gt;&gt; 4) Make an ifdef that lets ports switch between KURL and GURL.<br>
&gt;&gt;<br>
&gt;&gt; Once we get to this point, it will be easy to evaluate the performance<br>
&gt;&gt; and correctness impact of switching from KURL to GURL.  Ideally, we&#39;d<br>
&gt;&gt; have a &quot;bake-off&quot; and pick the best parser based on some objective<br>
&gt;&gt; criterion.  If KURL wins the bake-off, we should consider refactoring<br>
&gt;&gt; KURL&#39;s parser not to depend on WebCore::String (or really any of<br>
&gt;&gt; WebCore).  If GURL wins the bake-off, we should document (either with<br>
&gt;&gt; comments or tests) which parts of GURL&#39;s API is needed by Chromium but<br>
&gt;&gt; not necessarily need by other ports.<br>
&gt;&gt;<br>
&gt;&gt; Adam<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; On Tue, Apr 13, 2010 at 9:17 PM, Chris Jerdonek &lt;<a href="mailto:cjerdonek@webkit.org" target="_blank">cjerdonek@webkit.org</a>&gt;<br>
&gt;&gt; wrote:<br>
&gt;&gt; &gt; Regarding the URL parsing code, could someone that attended the<br>
&gt;&gt; &gt; session list what steps were proposed or tentatively agreed to (of<br>
&gt;&gt; &gt; which the below is the first)?<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Thanks a lot,<br>
&gt;&gt; &gt; --Chris<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; On Tue, Apr 13, 2010 at 1:46 AM, Adam Barth &lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt; wrote:<br>
&gt;&gt; &gt;&gt; Have you ever wanted WebKit&#39;s URL parsing to be awesome?  Do crazy<br>
&gt;&gt; &gt;&gt; characters in URLs keep you up at night?  Do you love writing tests?<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; If you answered yes to any of these questions, you might want to join<br>
&gt;&gt; &gt;&gt; the URL hackathon.  In this hackathon, we&#39;re adding a ton of test of<br>
&gt;&gt; &gt;&gt; our URL parsing code by adapting a BSD-licensed unit test suit into a<br>
&gt;&gt; &gt;&gt; set of LayoutTests.<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; You can find the instructions and sign-up sheet at following URL:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; <a href="http://docs.google.com/Doc?docid=0AZpchfQ5mBrEZGQ0cDh3YzRfMTVnZmZ3dGNmNg&amp;hl=en" target="_blank">http://docs.google.com/Doc?docid=0AZpchfQ5mBrEZGQ0cDh3YzRfMTVnZmZ3dGNmNg&amp;hl=en</a><br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; Thanks, and happy hacking!<br>
&gt;&gt; &gt;&gt; Adam<br>
&gt;&gt; &gt;&gt; _______________________________________________<br>
&gt;&gt; &gt;&gt; webkit-dev mailing list<br>
&gt;&gt; &gt;&gt; <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
&gt;&gt; &gt;&gt; <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; webkit-dev mailing list<br>
&gt;&gt; <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
&gt;&gt; <a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
&gt;<br>
&gt;<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>
</div></div></blockquote></div></div></div><br>
</blockquote></div><br>