[Webkit-unassigned] [Bug 54928] New: PrettyPatch.rb doesn't handle diffs where lines matching /^--/ are missing properly

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=54928

           Summary: PrettyPatch.rb doesn't handle diffs where lines
                    matching /^--/ are missing properly
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Mac OS X 10.5
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: dpranke at chromium.org
                CC: aroben at apple.com, ojan at chromium.org


This bug is cloned from bug 54918 - the test that was causing the problem has been patched, but this bug will track the root cause.

>From bug 54918:
> 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.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.