[Webkit-unassigned] [Bug 137586] import-w3c-test rewrites relative src paths in ref files incorrectly
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Oct 15 14:00:59 PDT 2014
https://bugs.webkit.org/show_bug.cgi?id=137586
--- Comment #5 from Rebecca Hauck <rhauck at adobe.com> 2014-10-15 14:00:54 PST ---
(From update of attachment 239821)
View in context: https://bugs.webkit.org/attachment.cgi?id=239821&action=review
>> Tools/Scripts/webkitpy/w3c/test_converter.py:170
>> + for attr in attrs:
>
> Why the switch here? It seems much more readable to use atter_name and attr_value instead of attr[0] and attr[1].
I mentioned this in my previous patch. I changed this to be more consistent with the other blocks in this function. If you think attr_name/value are better, I can change the other blocks in this function so they are all consistent. what do you think?
>> Tools/Scripts/webkitpy/w3c/test_converter.py:171
>> if attr_name == 'src':
>
> attr_name isn't defined.
oops, missed that. I'll fix in next patch.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list