<div>Thanks Dan for reply.</div><br><div class="gmail_quote">2010/6/2 Dan Bernstein <span dir="ltr"><<a href="mailto:mitz@apple.com">mitz@apple.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div style="word-wrap:break-word">Hi Xianzhu,<div><br></div><div><div><div class="im"><div>On Jun 1, 2010, at 9:37 PM, Xianzhu Wang wrote:</div><br><blockquote type="cite">Hi,<div><br></div><div>I'm new to webkit development, and I'd like to hear opinions about the problems I met.</div>
<div><br></div><div>Now I'm trying to fix some "old" layout bugs, for example:</div><div>
* white space preceding <br> (<a href="https://bugs.webkit.org/show_bug.cgi?id=37261" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=37261</a>)</div><div> * relative font-size (<a href="https://bugs.webkit.org/show_bug.cgi?id=18413" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=18413</a>)</div>
<div> * line breaking around some punctuations (<a href="https://bugs.webkit.org/show_bug.cgi?id=37698" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=37698</a>)</div><div><br></div><div>Some people have warned me about the difficulty of fixing these bugs, and now I have realized it. Fixing the bugs themselves is not very difficult, for example, only 2 functional lines change can fix the first bug. However, the change will break more than 4000 existing layout tests mostly because trailing spaces preceding <br>s in current expectations of the tests (for example, "PASS " vs "PASS"). I tried to rebaseline all effected layout tests (for now on mac only), and the patch is about 6MB (Sorry I overlooked the "Bigfile" option when I submitted the patch, so I split it into 4 parts).</div>
<div><br></div><div>My questions are:</div><div><br></div><div>1. The bugs violate the standards and cause some site compatibility issues. However, because the bugs are old, some web developers might treat them as features and depend on them, so fixing them might break some existing pages. Is there any existing policy about this problem?</div>
</blockquote><div><br></div></div><div>Not sure. I think WebKit should not make a policy decision to diverge from the standard (quirks mode withstanding) unless WebKit developers are actively advocating a change to the relevant standard.</div>
<div class="im"><br><blockquote type="cite"><div> Are these bugs worth fixing?</div></blockquote><div><br></div></div><div><div class="im"><div> * white space preceding <br> (<a href="https://bugs.webkit.org/show_bug.cgi?id=37261" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=37261</a>)</div>
<div><div><br></div></div></div><div>Unless I’m missing something, there is no known downside to fixing this bug. Seems worthwhile.</div><div class="im"><div><br></div><div><div><div> * relative font-size (<a href="https://bugs.webkit.org/show_bug.cgi?id=18413" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=18413</a>)</div>
<br></div></div></div><div>Is this really a standards compliance issue or just compatibility with Firefox? If there is a standards issue, is there a subset of the change that’s sufficient for standards compliance? Unless WebKit is in violation of a standard here, I personally don’t consider this important, but I know that other contributors have agreed to this change in principle, and it’s just that the patch has been abandoned by the original author.</div>
</div></div></div></div></blockquote><div><br></div><div>Yes, this is a standard compliance issue (<a href="http://www.w3.org/TR/CSS2/fonts.html#font-size-props">http://www.w3.org/TR/CSS2/fonts.html#font-size-props</a>). I have asked the original author and he'd like me to continue his work.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word"><div><div><div><div class="im"><div><br></div><div><div><div> * line breaking around some punctuations (<a href="https://bugs.webkit.org/show_bug.cgi?id=37698" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=37698</a>)</div>
</div></div><div><div><br></div></div></div><div>Seems worthwhile if it aligns WebKit with Firefox and IE. If moves away from IE compatibility, I wouldn’t do it. The main issue here is efficiency.</div></div></div><br></div>
</div></blockquote><div><br></div><div>I've uploaded another patch (for preview) that should have the same efficiency as the current algorithm and keep IE compatibility. I'll upload the whole patch soon.</div><div>
<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div style="word-wrap:break-word"><div></div><div>Personally, I think it’s good that you’re looking into such long-standing standards/compatibility issues!</div>
<div><br></div><div>Regards,</div><div>—Dan</div></div></blockquote></div><br>