BTW I did figure out why (code generators behave differently for overloaded and non-overloaded functions), mentioned that bug in the comments further down, and uploaded a patch to fix it. I'm sorry I didn't have had time to take it further.<br>
<br><div class="gmail_quote">On Wed, Aug 31, 2011 at 16:28, David Levin <span dir="ltr"><<a href="mailto:levin@google.com">levin@google.com</a>></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 Wed, Aug 31, 2011 at 5:57 AM, Philippe Normand <span dir="ltr"><<a href="mailto:philn@igalia.com" target="_blank">philn@igalia.com</a>></span> wrote:<br></div><div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Sorry for the last one. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Looks like the patch in <a href="https://bugs.webkit.org/show_bug.cgi?id=66045" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=66045</a><br>
is intended to fix this issue for both JSC and V8 code generators, if I<br>
understood correctly.<br></blockquote><div><br></div></div><div>Short story: <a href="http://trac.webkit.org/changeset/94023" style="font-size:13px;color:rgb(17, 85, 204);font-family:arial, sans-serif;background-color:rgb(255, 255, 255)" target="_blank">http://trac.webkit.org/<u></u>changeset/94023</a> is correct and we should fix the result that in there now.</div>


<div><br></div><div>More:</div><div>In general, when JSC and V8 disagree, we've gone with JSC in the platform independent file.</div><div><br></div><div><a href="http://trac.webkit.org/changeset/93713" target="_blank">http://trac.webkit.org/changeset/93713</a> and <a href="http://trac.webkit.org/changeset/94168" style="font-size:13px;color:rgb(17, 85, 204);font-family:arial, sans-serif;background-color:rgb(255, 255, 255)" target="_blank">http://trac.webkit.org/<u></u>changeset/94168</a> didn't do this so they aren't following what is standard practice.</div>


<div><br></div><div>The original issue crept in with r93713 which changed this output for v8 but didn't figure out why -- "<span style="font-family:monospace;white-space:pre-wrap;background-color:rgb(255, 255, 255)">No idea as to why the code the code generator acts differently now"</span>.</div>


<div><br></div><div>dave</div><div><div></div><div class="h5"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span><font color="#888888"><br>
Philippe<br>
</font></span><div><div></div><div><br>
On Wed, 2011-08-31 at 13:40 +0200, Osztrogonac Csaba wrote:<br>
> Hi,<br>
><br>
> It seems you guys are playing ping-pong with this platform independent expected file.<br>
> Could you decide which one is the correct?<br>
><br>
> br,<br>
> Ossy<br>
><br>
> <a href="http://trac.webkit.org/changeset/93713" target="_blank">http://trac.webkit.org/changeset/93713</a><br>
> <a href="http://trac.webkit.org/changeset/93713/trunk/LayoutTests/fast/files/create-blob-url-crash-expected.txt" target="_blank">http://trac.webkit.org/changeset/93713/trunk/LayoutTests/fast/files/create-blob-url-crash-expected.txt</a><br>



> -PASS: Not enough arguments<br>
> +PASS: Type error<br>
><br>
> <a href="http://trac.webkit.org/changeset/94023" target="_blank">http://trac.webkit.org/changeset/94023</a><br>
> <a href="http://trac.webkit.org/changeset/94023/trunk/LayoutTests/fast/files/create-blob-url-crash-expected.txt" target="_blank">http://trac.webkit.org/changeset/94023/trunk/LayoutTests/fast/files/create-blob-url-crash-expected.txt</a><br>



> -PASS: Type error<br>
> +PASS: Not enough arguments<br>
><br>
> <a href="http://trac.webkit.org/changeset/94168" target="_blank">http://trac.webkit.org/changeset/94168</a><br>
> <a href="http://trac.webkit.org/changeset/94168/trunk/LayoutTests/fast/files/create-blob-url-crash-expected.txt" target="_blank">http://trac.webkit.org/changeset/94168/trunk/LayoutTests/fast/files/create-blob-url-crash-expected.txt</a><br>



> -PASS: Not enough arguments<br>
> +PASS: Type error<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>
<br>
<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><br clear="all"><div><br></div>-- <br><font class="Apple-style-span" size="1" face="'arial narrow', sans-serif" color="#999999">Tommy Widenflycht, Senior Software Engineer<br>Google Sweden AB, Kungsbron 2, SE-11122 Stockholm, Sweden<br>
Org. nr. 556656-6880<br>And yes, I have to include the above in every outgoing email according to EU law.</font><br>