| Index: bindings/dart/scripts/dart_dictionary.py
|
| diff --git a/bindings/scripts/v8_dictionary.py b/bindings/dart/scripts/dart_dictionary.py
|
| similarity index 66%
|
| copy from bindings/scripts/v8_dictionary.py
|
| copy to bindings/dart/scripts/dart_dictionary.py
|
| index 6dc772fdcf99944f5204b50dc7bcc8302e07093a..42a21ba1725b0a15f10969b1d5c20094daee82b7 100644
|
| --- a/bindings/scripts/v8_dictionary.py
|
| +++ b/bindings/dart/scripts/dart_dictionary.py
|
| @@ -6,11 +6,10 @@
|
| implementation classes that are used by blink's core/modules.
|
| """
|
|
|
| -import copy
|
| import operator
|
| from v8_globals import includes
|
| -import v8_types
|
| -import v8_utilities
|
| +import dart_types
|
| +from dart_utilities import DartUtilities
|
|
|
|
|
| DICTIONARY_H_INCLUDES = frozenset([
|
| @@ -19,45 +18,46 @@ DICTIONARY_H_INCLUDES = frozenset([
|
| ])
|
|
|
| DICTIONARY_CPP_INCLUDES = frozenset([
|
| + 'bindings/common/ExceptionState.h',
|
| # FIXME: Remove this, http://crbug.com/321462
|
| 'bindings/core/v8/Dictionary.h',
|
| ])
|
|
|
|
|
| def setter_name_for_dictionary_member(member):
|
| - return 'set%s' % v8_utilities.capitalize(member.name)
|
| + return 'set%s' % DartUtilities.capitalize(member.name)
|
|
|
|
|
| def has_method_name_for_dictionary_member(member):
|
| - return 'has%s' % v8_utilities.capitalize(member.name)
|
| + return 'has%s' % DartUtilities.capitalize(member.name)
|
|
|
|
|
| -# Context for V8 bindings
|
| +def unwrap_nullable_if_needed(idl_type):
|
| + if idl_type.is_nullable:
|
| + return idl_type.inner_type
|
| + return idl_type
|
| +
|
| +
|
| +# Context for Dart bindings
|
|
|
| def dictionary_context(dictionary):
|
| includes.clear()
|
| includes.update(DICTIONARY_CPP_INCLUDES)
|
| return {
|
| - 'cpp_class': v8_utilities.cpp_name(dictionary),
|
| + 'cpp_class': DartUtilities.cpp_name(dictionary),
|
| 'header_includes': set(DICTIONARY_H_INCLUDES),
|
| 'members': [member_context(member)
|
| for member in sorted(dictionary.members,
|
| key=operator.attrgetter('name'))],
|
| - 'v8_class': v8_utilities.v8_class_name(dictionary),
|
| + 'dart_class': dart_types.dart_type(dictionary.name),
|
| +
|
| }
|
|
|
|
|
| def member_context(member):
|
| idl_type = member.idl_type
|
| idl_type.add_includes_for_type()
|
| -
|
| - def idl_type_for_default_value():
|
| - copied_type = copy.copy(idl_type)
|
| - # IdlType for default values shouldn't be nullable. Otherwise,
|
| - # it will generate meaningless expression like
|
| - # 'String("default value").isNull() ? ...'.
|
| - copied_type.is_nullable = False
|
| - return copied_type
|
| + idl_type = unwrap_nullable_if_needed(idl_type)
|
|
|
| def default_values():
|
| if not member.default_value:
|
| @@ -65,24 +65,30 @@ def member_context(member):
|
| if member.default_value.is_null:
|
| return None, 'v8::Null(isolate)'
|
| cpp_default_value = str(member.default_value)
|
| - v8_default_value = idl_type_for_default_value().cpp_value_to_v8_value(
|
| + v8_default_value = idl_type.cpp_value_to_v8_value(
|
| cpp_value=cpp_default_value, isolate='isolate',
|
| creation_context='creationContext')
|
| return cpp_default_value, v8_default_value
|
|
|
| - cpp_default_value, v8_default_value = default_values()
|
| + cpp_default_value, dart_default_value = default_values()
|
| +
|
| + dart_enum_expression = idl_type.enum_validation_expression
|
| + if dart_enum_expression:
|
| + dart_enum_expression = dart_enum_expression.format(param_name='string')
|
|
|
| return {
|
| 'cpp_default_value': cpp_default_value,
|
| 'cpp_type': idl_type.cpp_type,
|
| - 'cpp_value_to_v8_value': idl_type.cpp_value_to_v8_value(
|
| - cpp_value='impl->%s()' % member.name, isolate='isolate',
|
| + 'cpp_value_to_dart_value': idl_type.cpp_value_to_dart_value(
|
| + cpp_value='impl->%s()' % member.name,
|
| creation_context='creationContext',
|
| extended_attributes=member.extended_attributes),
|
| + 'enum_validation_expression': dart_enum_expression,
|
| 'has_method_name': has_method_name_for_dictionary_member(member),
|
| + 'is_object': idl_type.name == 'Object',
|
| 'name': member.name,
|
| 'setter_name': setter_name_for_dictionary_member(member),
|
| - 'v8_default_value': v8_default_value,
|
| + 'dart_default_value': dart_default_value,
|
| }
|
|
|
|
|
| @@ -93,7 +99,7 @@ def dictionary_impl_context(dictionary, interfaces_info):
|
| header_includes = set(['platform/heap/Handle.h'])
|
| return {
|
| 'header_includes': header_includes,
|
| - 'cpp_class': v8_utilities.cpp_name(dictionary),
|
| + 'cpp_class': DartUtilities.cpp_name(dictionary),
|
| 'members': [member_impl_context(member, interfaces_info,
|
| header_includes)
|
| for member in dictionary.members],
|
| @@ -101,7 +107,8 @@ def dictionary_impl_context(dictionary, interfaces_info):
|
|
|
|
|
| def member_impl_context(member, interfaces_info, header_includes):
|
| - idl_type = member.idl_type
|
| + idl_type = unwrap_nullable_if_needed(member.idl_type)
|
| + is_object = idl_type.name == 'Object'
|
|
|
| def getter_expression():
|
| if idl_type.impl_should_use_nullable_container:
|
| @@ -109,23 +116,30 @@ def member_impl_context(member, interfaces_info, header_includes):
|
| return 'm_%s' % member.name
|
|
|
| def has_method_expression():
|
| - if (idl_type.impl_should_use_nullable_container or
|
| - idl_type.is_string_type):
|
| + if idl_type.impl_should_use_nullable_container or idl_type.is_enum or idl_type.is_string_type:
|
| return '!m_%s.isNull()' % member.name
|
| + elif is_object:
|
| + return '!(m_{0}.isEmpty() || m_{0}.isNull() || m_{0}.isUndefined())'.format(member.name)
|
| else:
|
| return 'm_%s' % member.name
|
|
|
| def member_cpp_type():
|
| member_cpp_type = idl_type.cpp_type_args(used_in_cpp_sequence=True)
|
| if idl_type.impl_should_use_nullable_container:
|
| - return v8_types.cpp_template_type('Nullable', member_cpp_type)
|
| + return dart_types.cpp_template_type('Nullable', member_cpp_type)
|
| return member_cpp_type
|
|
|
| + cpp_default_value = None
|
| + if member.default_value and not member.default_value.is_null:
|
| + cpp_default_value = str(member.default_value)
|
| +
|
| header_includes.update(idl_type.impl_includes_for_type(interfaces_info))
|
| return {
|
| + 'cpp_default_value': cpp_default_value,
|
| 'getter_expression': getter_expression(),
|
| 'has_method_expression': has_method_expression(),
|
| 'has_method_name': has_method_name_for_dictionary_member(member),
|
| + 'is_object': is_object,
|
| 'is_traceable': (idl_type.is_garbage_collected or
|
| idl_type.is_will_be_garbage_collected),
|
| 'member_cpp_type': member_cpp_type(),
|
|
|