LayoutTests/platform/mac/fast/objc/dom-html-select-activate.html has a few statements of the form log("-- foo") in it, which causes a text line containing two dashes at the start of the line to be output in dumpAsText() mode. If the test fails in such a way that those lines aren't output (as it does on chromium-mac at the moment), then the unified diff contains a line with "---" at the start. PrettyPatch.rb interprets that as the start of a new file for diffing, and gets confused when there's no filename, and errors out. I think, in FileDiff.parse, it needs to be taught to understand the "@@ -1,26 +1,6 @@" line and skip over the appropriate number of lines of input correctly, rather than just looking for a line containing "---". My Ruby is weak, and I am lazy, and PrettyPatch appears to have no unit tests, so I haven't actually submitted a patch to do this. If, however, you change dom-html-select-activate to the contents of the first attachment, you'll get the diff in the second attachment, and old-run-webkit-tests will keel over with PrettyPatch failing. In the meantime, we could also just change the test and the expected output to not contain lines starting with "--" :). I'll attach a patch to do that as well.
Created attachment 83244 [details] sample changed version of the test file that will cause PrettyPatch to crash
Created attachment 83245 [details] the diff produced by the test in the first attachment. PrettyPatch will crash on this directly.
Created attachment 83247 [details] Patch
Comment on attachment 83247 [details] Patch I'd eventually like to see prettypatch fixed. Can you file a separate bug for that and make this bug specifically about the layout test?
(In reply to comment #4) > (From update of attachment 83247 [details]) > I'd eventually like to see prettypatch fixed. Can you file a separate bug for that and make this bug specifically about the layout test? Sure.
Committed r79281: <http://trac.webkit.org/changeset/79281>