<div class="gmail_quote">On Mon, Jun 1, 2009 at 11:30 PM, Adam Barth <span dir="ltr">&lt;<a href="mailto:abarth@webkit.org">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;">
<div class="im">On Mon, Jun 1, 2009 at 8:29 PM, Jeremy Orlow &lt;<a href="mailto:jorlow@chromium.org">jorlow@chromium.org</a>&gt; wrote:<br>
&gt; If this is the only issue, the parsing code could work around it.  There are<br>
&gt; 3 parts to the identifier: the protocol (should never have a _ in it,<br>
&gt; right?), the domain, and the port (once again, should never have a _).  It<br>
&gt; seems as though the parsing code should look for the first _, the last _,<br>
&gt; and then assume everything in the middle is the domain.<br>
<br>
</div>I don&#39;t know of any reason why a scheme can&#39;t have a _...  If you&#39;d<br>
like to change the parsing code to understand domains with a _ this<br>
way,  I think that would be an improvement.<br>
<div class="im"><br>
&gt; Doesn&#39;t seem like a top priority, but maybe it&#39;s worth filing a low priority<br>
&gt; bug for it anyway.  Although they are 2 somewhat distinct use cases and I<br>
&gt; see the possibility for misuse and bad assumptions, I&#39;m not terribly worried<br>
&gt; about it.<br>
<br>
</div>I think HTML 5 has notions of &quot;origin&quot; and &quot;effective script origin&quot;<br>
(or some such) that separate these two concepts.  It might be worth<br>
syncing up our internal names with the spec to make these concepts<br>
more accessible to future developers.</blockquote><div><br></div><div>Agreed.  Most of the new features use the simpler same origin policy which compares only the protocol, port, and domain.  The effective script origin  and the security around cookies are the more complex parts.  I believe there&#39;s a fairly clean split between the two parts in the source code.  I&#39;ll file a bug tomorrow regarding this opportunity for cleanup.</div>
<div><br></div><div>J</div></div>