<br><br><div class="gmail_quote">On Mon, Jan 16, 2012 at 12:02 AM, Gyuyoung Kim <span dir="ltr"><<a href="mailto:gyuyoung@gmail.com">gyuyoung@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>Hello WebKit folks.</div><div><br></div><div>It looks that the result of layout test for EFL port needs rebaseline because of 101343. The revision modified line spacing of font in SimpleFontDataFreeType.cpp, and it seems that the existing layout test result of EFL port was influenced by it. (<a href="http://trac.webkit.org/changeset/101343" target="_blank">http://trac.webkit.org/changeset/101343</a>)</div>

<div><br></div><div>3,936 test cases are failed after the revision (101343). </div><div><br></div><div>    * EFL port's layout test result with r101343</div><div><br></div><div>      => Results: 16139/27915 tests passed (57.8%)</div>

<div>      => Tests to be fixed (11776):</div><div>        4423 text diff mismatch       (37.6%)</div><div>        9 image mismatch           ( 0.1%)</div><div>        7344 skipped                  (62.4%)</div><div><br>

</div><div><br></div><div>    * EFL port's layout test Result without r101343</div><div><br></div><div>      => Results: 20074/27915 tests passed (71.9%)</div><div>      => Tests to be fixed (7841):</div><div>        1 test timed out           ( 0.0%)</div>

<div>        487 text diff mismatch       ( 6.2%)</div><div>        9 image mismatch           ( 0.1%)</div><div>        7344 skipped                  (93.7%)</div><div><br></div><div><br></div><div>It looks GTK port also did rebaseline due to the revision.</div>

<div> - <a href="http://trac.webkit.org/changeset/101354" target="_blank">http://trac.webkit.org/changeset/101354</a></div><div> - <a href="http://trac.webkit.org/changeset/101352" target="_blank">http://trac.webkit.org/changeset/101352</a></div>
<div> - <a href="http://trac.webkit.org/changeset/101351" target="_blank">http://trac.webkit.org/changeset/101351</a></div>
<div> - And so on.</div><div><br></div><div>I think EFL port also needs to do rebaseline for layout test result. GTK port did rebaseline without review because patch is too huge. Is there any processes for rebaseline ?</div>
</blockquote><div><br></div><div>In Chromium, rebaselines are nearly always landed by Chromium committers without review.  If there is some doubt on whether a rebaseline is correct or not it's expected that the committer will check with the appropriate party, but we don't require the full review process.</div>
<div><br></div><div>I'd expect other ports follow a similar guideline for their own baselines.</div><div><br></div><div>- James</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888">
<div><br></div><div>- gyuyoung</div>
</font></span><br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">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></blockquote></div><br>