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

Unified Diff: Source/bindings/scripts/unstable/v8_callback_interface.py

Issue 31933002: IDL compiler: [SetterCallWith] (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Revised Created 7 years, 2 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/unstable/v8_attributes.py ('k') | Source/bindings/scripts/unstable/v8_utilities.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/bindings/scripts/unstable/v8_callback_interface.py
diff --git a/Source/bindings/scripts/unstable/v8_callback_interface.py b/Source/bindings/scripts/unstable/v8_callback_interface.py
index 1d70ff058b1156bc53294ad1011f8e522e591efe..89e256618e8b475631a98fe130d4552dc634681e 100644
--- a/Source/bindings/scripts/unstable/v8_callback_interface.py
+++ b/Source/bindings/scripts/unstable/v8_callback_interface.py
@@ -36,7 +36,7 @@ For details, see bug http://crbug.com/239771
"""
from v8_types import cpp_type, cpp_value_to_v8_value, includes_for_type
-from v8_utilities import v8_class_name, has_extended_attribute_value
+from v8_utilities import v8_class_name, extended_attribute_value_contains
CALLBACK_INTERFACE_H_INCLUDES = set([
'bindings/v8/ActiveDOMCallback.h',
@@ -94,12 +94,13 @@ def includes_for_operation(operation):
def generate_method_contents(operation):
- call_with_this_handle = has_extended_attribute_value(operation.extended_attributes, 'CallWith', 'ThisValue')
+ extended_attributes = operation.extended_attributes
+ call_with = extended_attributes.get('CallWith')
contents = {
'name': operation.name,
'return_cpp_type': cpp_type(operation.data_type, 'RefPtr'),
- 'custom': 'Custom' in operation.extended_attributes,
- 'call_with_this_handle': call_with_this_handle,
+ 'custom': 'Custom' in extended_attributes,
+ 'call_with_this_handle': extended_attribute_value_contains(call_with, 'ThisValue'),
}
contents.update(generate_arguments_contents(operation.arguments, call_with_this_handle))
return contents
« no previous file with comments | « Source/bindings/scripts/unstable/v8_attributes.py ('k') | Source/bindings/scripts/unstable/v8_utilities.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698