Hi,<div><br></div><div>Any update on this?</div><div><br></div><div>Is it OK to remove &#39;stroke=&quot;#0&quot;&#39; from:</div><div>  LayoutTests/svg/custom/invalid-fill-hex.svg</div><div><div>    &lt;rect width=&quot;100&quot; height=&quot;100&quot; fill=&quot;#ff&quot; stroke=&quot;#0&quot;/&gt;</div>

</div><div>?</div><div><br></div><div>If there are no objections, I&#39;ll upload a patch to remove it for review.</div><div><br></div><div>Yuzo</div><div><br></div><div><br></div><div><div class="gmail_quote">On Tue, Jun 15, 2010 at 5:52 PM, Yuzo Fujishima <span dir="ltr">&lt;<a href="mailto:yuzo@google.com">yuzo@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;">Nikolas,<div><br></div><div>Thank you for your response.</div><div>I&#39;ll hold off committing <span style="font-family:arial, sans-serif;font-size:13px;border-collapse:collapse;color:rgb(68, 68, 68)"><a href="https://bugs.webkit.org/show_bug.cgi?id=24742" style="color:rgb(34, 34, 34)" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=24742</a> until</span></div>


<div><font color="#444444" face="arial, sans-serif"><span style="border-collapse:collapse">you make a decision. Please let me know when you make one.</span></font></div>
<div><font color="#444444" face="arial, sans-serif"><span style="border-collapse:collapse"><br></span></font></div><div><font color="#888888"><font color="#444444" face="arial, sans-serif"><span style="border-collapse:collapse">Yuzo<br>


</span></font></font><div><div></div><div class="h5"><div><br><div class="gmail_quote">On Tue, Jun 15, 2010 at 3:42 PM, Nikolas Zimmermann <span dir="ltr">&lt;<a href="mailto:zimmermann@physik.rwth-aachen.de" target="_blank">zimmermann@physik.rwth-aachen.de</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Am 15.06.2010 um 08:28 schrieb Cameron McCormack:<div><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Nikolas Zimmermann:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We had &#39;none&#39; for years, until compatibility bugs were raised, as<br>
Opera uses black as default stroke, if there is any error.<br>
</blockquote>
<br>
Doesn’t seem to be the case any more, at least for Opera 10.53:<br>
<br>
<a href="http://livedom.validator.nu/?%3C!DOCTYPE%20html%3E%0D%0A%3Csvg%20width%3D&#39;100&#39;%20height%3D&#39;100&#39;%3E%3Ccircle%20cx%3D&#39;50&#39;%20cy%3D&#39;50&#39;%20r%3D&#39;30&#39;%20fill%3D&#39;yellow&#39;%20stroke%3D&#39;xxx&#39;%20stroke-width%3D&#39;4&#39;%2F%3E%3C%2Fsvg%3E" target="_blank">http://livedom.validator.nu/?%3C!DOCTYPE%20html%3E%0D%0A%3Csvg%20width%3D&#39;100&#39;%20height%3D&#39;100&#39;%3E%3Ccircle%20cx%3D&#39;50&#39;%20cy%3D&#39;50&#39;%20r%3D&#39;30&#39;%20fill%3D&#39;yellow&#39;%20stroke%3D&#39;xxx&#39;%20stroke-width%3D&#39;4&#39;%2F%3E%3C%2Fsvg%3E</a><br>



<br>
Note the stroke=&quot;xxx&quot;.  Firefox nightlies behave the same.<br>
</blockquote>
<br></div>
That would be good news, I&#39;ve been against the switch all time, we should reiterate that specific behaviour. Turning off that default stroke behaviour and seeing which layout tests break, and whether we now consider this a progression :-)<br>



I&#39;ll have a look at it this week.<br>
<br>
Cheers,<br>
Niko<div><div></div><div><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><br></div></div></div></div>
</blockquote></div><br></div>