Fix V8 bindings for named constructors to set prototype object correctly

third_party/WebKit/Source/bindings/templates/attributes.cpp.tmpl

 {% from 'utilities.cpp.tmpl' import declare_enum_validation_variable, v8_value_to_local_cpp_value %}
 
 {##############################################################################}
 {% macro attribute_getter(attribute, world_suffix) %}
 static void {{attribute.name}}AttributeGetter{{world_suffix}}(
 {%- if attribute.is_data_type_property %}
 const v8::PropertyCallbackInfo<v8::Value>& info
 {%- else %}
 const v8::FunctionCallbackInfo<v8::Value>& info
 {%- endif %}) {
@@ skipping 408 matching lines @@
 {% endmacro %}
 
 
 {##############################################################################}
 {% macro attribute_configuration(attribute) %}
 {% from 'utilities.cpp.tmpl' import property_location %}
 {% if attribute.constructor_type %}
 {% set getter_callback =
        '%sV8Internal::%sConstructorGetterCallback' % (cpp_class_or_partial, attribute.name)
        if attribute.needs_constructor_getter_callback else
-       'v8ConstructorAttributeGetter' %}
+       ('V8%s::NamedConstructorAttributeGetter' % (attribute.constructor_type)
+        if attribute.is_named_constructor else
+        'v8ConstructorAttributeGetter') %}
 {% set setter_callback = '0' %}
 {% else %}{# regular attributes #}
 {% set getter_callback = '%sV8Internal::%sAttributeGetterCallback' %
            (cpp_class_or_partial, attribute.name) %}
 {% set setter_callback = '%sV8Internal::%sAttributeSetterCallback' %
            (cpp_class_or_partial, attribute.name)
        if attribute.has_setter else '0' %}
 {% endif %}
 {% set getter_callback_for_main_world =
        '%sForMainWorld' % getter_callback
        if attribute.is_per_world_bindings else '0' %}
 {% set setter_callback_for_main_world =
        '%sForMainWorld' % setter_callback
        if attribute.is_per_world_bindings and attribute.has_setter else '0' %}
 {% set wrapper_type_info =
        'const_cast<WrapperTypeInfo*>(&V8%s::wrapperTypeInfo)' %
            attribute.constructor_type
        if attribute.constructor_type else '0' %}

[Yuki, 2017/01/20 05:54:42]

nit: Since you no longer need |data| in NamedConstructorAttributeGetter, you
don't need this wrapper type info, too.  You can get rid of this if it's named
constructor.

[sashab, 2017/01/25 02:34:51]

Done, I think. Is this what you meant?

 {% if attribute.is_data_type_property %}
 {% set access_control = 'static_cast<v8::AccessControl>(%s)' %
                         ' | '.join(attribute.access_control_list) %}
 {% else %}
 {% set access_control = 'v8::DEFAULT' %}
 {% endif %}
 {% set property_attribute = 'static_cast<v8::PropertyAttribute>(%s)' %
                             ' | '.join(attribute.property_attributes) %}
 {% set cached_accessor_callback =
        '%sV8Internal::%sCachedAccessorCallback' % (cpp_class_or_partial, attribute.name)
        if attribute.is_cached_accessor else
        'nullptr' %}
 {% set holder_check = 'V8DOMConfiguration::DoNotCheckHolder'
        if attribute.is_lenient_this else 'V8DOMConfiguration::CheckHolder' %}
 {% set attribute_configuration_list = [
        '"%s"' % attribute.name,
        getter_callback,
        setter_callback,
        getter_callback_for_main_world,
        setter_callback_for_main_world,
        cached_accessor_callback,
        wrapper_type_info,
        access_control,
        property_attribute,
        property_location(attribute),
        holder_check,
    ] %}
 {{'{'}}{{attribute_configuration_list | join(', ')}}{{'}'}}
 {%- endmacro %}