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

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

Issue 328663003: IDL: restructure logic handling registration of methods (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 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
Index: Source/bindings/scripts/v8_methods.py
diff --git a/Source/bindings/scripts/v8_methods.py b/Source/bindings/scripts/v8_methods.py
index b50ba604efeda5d30d6904c14ab2371aec03b82c..d14bfa0c0e95bf11c35c30775916365eef8d4fda 100644
--- a/Source/bindings/scripts/v8_methods.py
+++ b/Source/bindings/scripts/v8_methods.py
@@ -40,6 +40,17 @@ import v8_utilities
from v8_utilities import has_extended_attribute_value
+# Methods with any of these require custom method registration code in the
+# interface's configure*Template() function.
+CUSTOM_REGISTRATION_EXTENDED_ATTRIBUTES = (
+ 'DoNotCheckSecurity',
+ 'DoNotCheckSignature',
+ 'NotEnumerable',
+ 'ReadOnly',
+ 'Unforgeable',
+)
+
+
def argument_needs_try_catch(argument):
idl_type = argument.idl_type
base_type = not idl_type.array_or_sequence_type and idl_type.base_type
@@ -107,10 +118,6 @@ def generate_method(interface, method):
'cpp_type': idl_type.cpp_type,
'cpp_value': this_cpp_value,
'deprecate_as': v8_utilities.deprecate_as(method), # [DeprecateAs]
- 'do_not_check_signature': not(is_static or
- v8_utilities.has_extended_attribute(method,
- ['DoNotCheckSecurity', 'DoNotCheckSignature', 'NotEnumerable',
- 'ReadOnly', 'RuntimeEnabled', 'Unforgeable'])),
'function_template': function_template(),
'idl_type': idl_type.base_type,
'has_event_listener_argument': has_event_listener_argument,
@@ -139,6 +146,8 @@ def generate_method(interface, method):
'is_variadic': arguments and arguments[-1].is_variadic,
'measure_as': v8_utilities.measure_as(method), # [MeasureAs]
'name': name,
+ 'needs_custom_registration': is_static or
+ v8_utilities.has_extended_attribute(method, CUSTOM_REGISTRATION_EXTENDED_ATTRIBUTES),
Nils Barth (inactive) 2014/06/11 10:10:28 nit: line break
'number_of_arguments': len(arguments),
'number_of_required_arguments': len([
argument for argument in arguments

Powered by Google App Engine
This is Rietveld 408576698