DumpRenderTree for the EFL port :)
Hey, At last, the EFL port of WebKit got a DumpRenderTree implementation! We're still working on ironing out a lot of bugs found by some of the LayoutTests, but the DRT (and ImageDiff) code has been submitted to Bugzilla. It would be awesome if anyone could help reviewing these patches. DumpRenderTree patches: https://bugs.webkit.org/show_bug.cgi?id=62034 https://bugs.webkit.org/show_bug.cgi?id=61974 https://bugs.webkit.org/show_bug.cgi?id=61962 https://bugs.webkit.org/show_bug.cgi?id=61958 ImageDiff patch: https://bugs.webkit.org/show_bug.cgi?id=61942 Thanks, Leandro
This is *so* importnat to the long-term health of your port. Congrats! I'm happy to review what I can and will look through them now. -eric On Fri, Jun 10, 2011 at 12:02 PM, Leandro Pereira <leandro@profusion.mobi> wrote:
Hey,
At last, the EFL port of WebKit got a DumpRenderTree implementation! We're still working on ironing out a lot of bugs found by some of the LayoutTests, but the DRT (and ImageDiff) code has been submitted to Bugzilla. It would be awesome if anyone could help reviewing these patches.
DumpRenderTree patches: https://bugs.webkit.org/show_bug.cgi?id=62034 https://bugs.webkit.org/show_bug.cgi?id=61974 https://bugs.webkit.org/show_bug.cgi?id=61962 https://bugs.webkit.org/show_bug.cgi?id=61958
ImageDiff patch: https://bugs.webkit.org/show_bug.cgi?id=61942
Thanks, Leandro _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
I looked through all the patches. There is a general trouble of that the code is written in very-old C-style yet uses a c++ compiler. I think this will come back to haunt you in the future as more people try to work on the code. I suspect that there are many memory leaks in the code as written, mostly due to the lack of smart pointers. I'm ready to r+ the patches re-posted with some modern c++ usage. I'm also OK with someone else approving them if EFL style requires this ancient-C (and bad memory-management) look. -eric On Fri, Jun 10, 2011 at 12:10 PM, Eric Seidel <eric@webkit.org> wrote:
This is *so* importnat to the long-term health of your port.
Congrats! I'm happy to review what I can and will look through them now.
-eric
On Fri, Jun 10, 2011 at 12:02 PM, Leandro Pereira <leandro@profusion.mobi> wrote:
Hey,
At last, the EFL port of WebKit got a DumpRenderTree implementation! We're still working on ironing out a lot of bugs found by some of the LayoutTests, but the DRT (and ImageDiff) code has been submitted to Bugzilla. It would be awesome if anyone could help reviewing these patches.
DumpRenderTree patches: https://bugs.webkit.org/show_bug.cgi?id=62034 https://bugs.webkit.org/show_bug.cgi?id=61974 https://bugs.webkit.org/show_bug.cgi?id=61962 https://bugs.webkit.org/show_bug.cgi?id=61958
ImageDiff patch: https://bugs.webkit.org/show_bug.cgi?id=61942
Thanks, Leandro _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Leandro Pereira <leandro@profusion.mobi> writes:
At last, the EFL port of WebKit got a DumpRenderTree implementation! We're still working on ironing out a lot of bugs found by some of the LayoutTests, but the DRT (and ImageDiff) code has been submitted to Bugzilla. It would be awesome if anyone could help reviewing these patches.
Thanks to eseidel, a few of our patchas have already made their way upstream. Unfortunately, he needs to take a break from too much exposure to C-style code, so we need someone else to review our remaining patches :-) The remaining DRT patches are: https://bugs.webkit.org/show_bug.cgi?id=61962 https://bugs.webkit.org/show_bug.cgi?id=61974 https://bugs.webkit.org/show_bug.cgi?id=63086 https://bugs.webkit.org/show_bug.cgi?id=61974 -- Raphael Kubo da Costa ProFUSION embedded systems http://profusion.mobi
Leandro Pereira <leandro@profusion.mobi> writes:
At last, the EFL port of WebKit got a DumpRenderTree implementation! We're still working on ironing out a lot of bugs found by some of the LayoutTests, but the DRT (and ImageDiff) code has been submitted to Bugzilla. It would be awesome if anyone could help reviewing these patches.
... And we finally finished upstreaming our work a few weeks ago, with baselines being committed some time later. We'd like to thank everyone, especially those who helped review our changes (eseidel, rniwa, tkent, tonikitoo, kenneth, mrobinson and so many others). Cheers! -- Raphael Kubo da Costa ProFUSION embedded systems http://profusion.mobi
participants (3)
-
Eric Seidel
-
Leandro Pereira
-
Raphael Kubo da Costa