I would say GTK's way is pretty acceptable in this case.<br><br><div class="gmail_quote">On Mon, Jan 16, 2012 at 3:21 AM, Gyuyoung Kim <span dir="ltr"><<a href="mailto:gyuyoung.kim@samsung.com">gyuyoung.kim@samsung.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello WebKit folks.<br>
<br>
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>)<br>


<br>
3,936 test cases are failed after the revision (101343).<br>
    * EFL port's layout test result with r101343<br>
      => Results: 16139/27915 tests passed (57.8%)<br>
      => Tests to be fixed (11776):<br>
        4423 text diff mismatch       (37.6%)<br>
        9 image mismatch           ( 0.1%)<br>
        7344 skipped                  (62.4%)<br>
<br>
    * EFL port's layout test Result without r101343<br>
      => Results: 20074/27915 tests passed (71.9%)<br>
      => Tests to be fixed (7841):<br>
        1 test timed out           ( 0.0%)<br>
        487 text diff mismatch       ( 6.2%)<br>
        9 image mismatch           ( 0.1%)<br>
        7344 skipped                  (93.7%)<br>
<br>
It looks GTK port also did rebaseline due to the revision.<br>
 - <a href="http://trac.webkit.org/changeset/101354" target="_blank">http://trac.webkit.org/changeset/101354</a><br>
 - <a href="http://trac.webkit.org/changeset/101352" target="_blank">http://trac.webkit.org/changeset/101352</a><br>
 - <a href="http://trac.webkit.org/changeset/101351" target="_blank">http://trac.webkit.org/changeset/101351</a><br>
 - And so on.<br>
<br>
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 ?<br>
<br>
- gyuyoung<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>
</blockquote></div><br><br clear="all"><br>-- <br>--Antonio Gomes<br>