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

[Yuki, 2017/01/19 09:57:28]

nit: Indent one more space?
    ('V8%s::...' % (...)
     if attribute... else
     '...')
because you have an open parenthesis.

[sashab, 2017/01/20 04:37:05]

Good idea -- done.

+       '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
@@ skipping 27 matching lines @@
        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 %}