Fixing the diff_parser to correctly identify git diffs even with leading comments.
Created attachment 184601 [details] Patch
Comment on attachment 184601 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=184601&action=review LGTM. > Tools/Scripts/webkitpy/common/checkout/diff_parser.py:78 > # FIXME: This method belongs on DiffParser > -def get_diff_converter(first_diff_line): > +def get_diff_converter(lines): Commonly to solve this sort of testing problem, we just make an object, make this function a method, and then do: def my_free_function(foo): return MyNewObject().my_free_function(foo) I guess we just haven't gotten to that with DiffParser yet.
Comment on attachment 184601 [details] Patch Clearing flags on attachment: 184601 Committed r140775: <http://trac.webkit.org/changeset/140775>
All reviewed patches have been landed. Closing bug.