I ran the layout tests with --pixel and --tolerance 0 just to verify that the baselines were in fact correct. These are my findings: I will upload new baselines for these two: editing/deleting/5390681-2.html - cursor isn't in expected image although it should be editing/pasteboard/5387578.html - collapse whitespace change in r42549 without new baselines For these tests, the expected images have slightly darker fonts than when I run them locally. Seems safe to ignore, but might be good to rebaseline. editing/deleting/type-delete-after-quote.html editing/inserting/5418891.html editing/selection/select-text-overflow-ellipsis.html editing/selection/transformed-selection-rects.html editing/inserting/4960120-1.html - Focus ring in the expected image is a bit darker (more blue). Seems safe to ignore, but might be good to rebaseline. editing/deleting/delete-to-select-table.html - one pixel off in the border, seems fine. editing/selection/select-all-002.html - one pixel off in the border, seems fine. editing/selection/4960116.html - the caret shows through even though the div that contains it is invisible. Maybe DRT is designed to always show the cursor? I don't see a caret in Safari. Seems safe to ignore, but maybe this is a bug in DRT. These tests appear to have regressed: editing/selection/caret-rtl-2.html - The test description says that the caret should be near where the user clicks, but it's not. Both the expected and the actual look wrong. Looking through the repo, I can't tell when this regressed. editing/selection/extend-by-word-002.html - The selection on my mac looks like two selections are overlapping (some of the selection is darker than other parts of the selection). I think this test has also regressed. Let me know if I should file separate bugs for any of these.
Created attachment 48204 [details] Patch
Comment on attachment 48204 [details] Patch Hmm, I though the images would show in the formatted diff. I will upload the images to a different server.
Result images and diffs are here: http://ponderer.org/tests/layout-test-results-20100204/
Comment on attachment 48204 [details] Patch Rejecting patch 48204 from commit-queue. Failed to run "['/Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', '--reviewer', 'Darin Adler', '--force']" exit_code: 255 patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/platform/mac/editing/deleting/5390681-2-expected.checksum only literal type is supported now at /Users/eseidel/Projects/CommitQueue/WebKitTools/Scripts/svn-apply line 283, <> line 751. Full output: http://webkit-commit-queue.appspot.com/results/238103
Attachment 48204 [details] was posted by a committer and has review+, assigning to Tony Chang for commit.
Committed r54475: <http://trac.webkit.org/changeset/54475>
I filed bug 34692 and bug 34693 for the 2 tests that seem to have regressed.