RESOLVED FIXED 30061
[Gtk][Layout tests] Generate results for editing tests
https://bugs.webkit.org/show_bug.cgi?id=30061
Summary [Gtk][Layout tests] Generate results for editing tests
Zan Dobersek
Reported 2009-10-04 11:20:35 PDT
Currently, about 730 tests in editing/ directory need to have results generated.
Attachments
Generated results (deleted)
2009-10-04 11:37 PDT, Zan Dobersek
no flags
Zan Dobersek
Comment 1 2009-10-04 11:37:02 PDT
Created attachment 40598 [details] Generated results This patch adds generated test results for about 350 tests. Only these tests produce 'rendering-equal' results to those from the mac port. With 'rendering-equal' phrase, these guarantees are meant: - Equal render objects are created in the same places (not necessarrily in the exact same position and with exact same size) - These objects have equal CSS attributes (borders, color etc.) - In the case of text runs, they contain equal text - Equal caret and selection attributes dump Basically, they're semantically equal, there can be differences in the size and position of render objects because of different font metrics. Tests that do not produce 'rendering-equal' results should be listed in another bug, examined and then fixed if fixable or added as generated if they're not fixable and acceptable as such.
Eric Seidel (no email)
Comment 2 2009-10-05 09:27:43 PDT
Comment on attachment 40598 [details] Generated results It is very sad that so many of these are render tree dumps instead of being dumpAsText (but that's not your fault). rubber-stamp=me.
Gustavo Noronha (kov)
Comment 3 2009-10-06 10:08:13 PDT
Comment on attachment 40598 [details] Generated results Setting cq+, since Zan is not a committer.
WebKit Commit Bot
Comment 4 2009-10-06 10:24:41 PDT
Comment on attachment 40598 [details] Generated results Clearing flags on attachment: 40598 Committed r49192: <http://trac.webkit.org/changeset/49192>
WebKit Commit Bot
Comment 5 2009-10-06 10:24:45 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.