Re: [webkit-dev] DumpRenderTree for the EFL port :)
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 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 > ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Re: [webkit-dev] DumpRenderTree for the EFL port :)
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 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 > 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 >> > ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Re: [webkit-dev] DumpRenderTree for the EFL port :)
Leandro Pereira 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 ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
Re: [webkit-dev] DumpRenderTree for the EFL port :)
Leandro Pereira 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 ___ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev