Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(347)

Unified Diff: Source/bindings/scripts/preprocess_idls.py

Issue 17214003: Remove [InterfaceName] IDL attribute (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix try bot failures Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/bindings/scripts/IDLAttributes.txt ('k') | Source/core/core.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/bindings/scripts/preprocess_idls.py
diff --git a/Source/bindings/scripts/preprocess_idls.py b/Source/bindings/scripts/preprocess_idls.py
index 8c5799dc6cacda7a50593eda2f655b3f892cd4c9..e9342121fcf7cdf95a4dfb7a36b31fa0831ef477 100644
--- a/Source/bindings/scripts/preprocess_idls.py
+++ b/Source/bindings/scripts/preprocess_idls.py
@@ -106,12 +106,7 @@ def generate_constructor_attribute_list(interface_name, extended_attributes):
else:
extended_string = ''
- # FIXME: Remove this once we remove [InterfaceName] IDL attribute, http://crbug.com/242137
- if 'InterfaceName' in extended_attributes:
- extended_interface_name = extended_attributes['InterfaceName']
- else:
- extended_interface_name = interface_name
- attribute_string = 'attribute %sConstructor %s' % (interface_name, extended_interface_name)
+ attribute_string = 'attribute %(interface_name)sConstructor %(interface_name)s' % {'interface_name': interface_name}
attributes_list = [extended_string + attribute_string]
# In addition to the regular property, for every [NamedConstructor]
« no previous file with comments | « Source/bindings/scripts/IDLAttributes.txt ('k') | Source/core/core.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698