<div dir="ltr"><div class="gmail_quote">On Thu, Oct 2, 2008 at 3:27 PM, David Hyatt <span dir="ltr">&lt;<a href="mailto:hyatt@apple.com">hyatt@apple.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="Ih2E3d">On Oct 2, 2008, at 4:23 PM, Maciej Stachowiak wrote:<br>
<br>
&gt;&gt;<br>
&gt;&gt; I have mentioned optionally replacing KURL with an ifdef to a number<br>
&gt;&gt; of WebKit members. The reception has been tentatively yes.<br>
&gt;<br>
&gt; As one of the people who were asked and tentatively said yes,<br>
<br>
</div>I am strongly against integration of GURL behind KURL. &nbsp;This code is<br>
simple, and there&#39;s no reason to complicate it like this. &nbsp;Any port<br>
should be able to use KURL as is and just translate at boundaries<br>
(like the existing Mac port does). &nbsp;I don&#39;t think there should be any<br>
major issues with this approach, and the alternative is to place a<br>
burden on WebKit as far as having to maintain a now needlessly<br>
complicated KURL class.<br></blockquote><div><br></div><div><br></div><div>Hi Dave,</div><div><br></div><div>Please see my comments on this thread. &nbsp;Summarizing: &nbsp;The code for correct and complete URL parsing / canonicalization is non-trivial and in many cases extremely subtle. &nbsp;It is important to Chromium to use consistent URL handling across the entire application. &nbsp;For example, our network stack depends on GoogleURL. &nbsp;We have strong desire to share the same URL handling code with WebCore so that we achieve consistency, avoiding both correctness and security bugs. &nbsp;It would be awkward to have our otherwise independent network stack suddenly depend on WebCore.</div>
<div><br></div><div>All we are asking for is to add a couple #ifdefs to KURL.h so that we do not have to maintain a fork. &nbsp;Is that really so hard to maintain going forward? &nbsp;We are happy to do all of the work to maintain it.</div>
<div><br></div><div>Regards,</div><div>-Darin</div></div></div>