Chromium Code Reviews| Index: Source/bindings/scripts/utilities.py |
| diff --git a/Source/bindings/scripts/utilities.py b/Source/bindings/scripts/utilities.py |
| index 28beb17c94e8a842c10f7f4ddafa980f47ef0224..367476598a1b8e0b115f749d41ce258941e91d78 100644 |
| --- a/Source/bindings/scripts/utilities.py |
| +++ b/Source/bindings/scripts/utilities.py |
| @@ -153,10 +153,10 @@ def get_interface_extended_attributes_from_idl(file_contents): |
| extended_attributes_string = match.group(1) |
| extended_attributes = {} |
| - # FIXME: this splitting is WRONG: it fails on ExtendedAttributeArgList like |
| - # 'NamedConstructor=Foo(a, b)' |
| + # FIXME: this splitting is WRONG: it fails on extended attributes where lists of |
| + # multiple values are used, which are seperated by a comma and a space. |
|
Peter Beverloo
2014/08/01 16:14:03
Are there plans to make these scripts use the actu
|
| parts = [extended_attribute.strip() |
| - for extended_attribute in extended_attributes_string.split(',') |
| + for extended_attribute in re.split(',\s+', extended_attributes_string) |
| # Discard empty parts, which may exist due to trailing comma |
| if extended_attribute.strip()] |
| for part in parts: |