bindings: Supports [Replaceable] accessors.

Source/bindings/templates/interface_base.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file has been auto-generated by {{code_generator}}. DO NOT MODIFY!
 
 #include "config.h"
 {% filter conditional(conditional_string) %}
 #include "{{v8_class_or_partial}}.h"
 
@@ skipping 33 matching lines @@
 {% endfor %}
 {% endif %}
 
 {# Constants #}
 {% from 'constants.cpp' import constant_getter_callback
        with context %}
 {% for constant in special_getter_constants %}
 {{constant_getter_callback(constant)}}
 {% endfor %}
 {# Attributes #}
+{% block replaceable_attribute_setter_and_callback %}
+{% if has_replaceable_attributes or has_constructor_attributes %}
+template<class CallbackInfo>
+static void {{cpp_class}}ForceSetAttributeOnThis(v8::Local<v8::String> name, v8::Local<v8::Value> v8Value, const CallbackInfo& info)
+{
+    {% if is_check_security %}
+    {{cpp_class}}* impl = {{v8_class}}::toImpl(info.Holder());
+    v8::String::Utf8Value attributeName(name);
+    ExceptionState exceptionState(ExceptionState::SetterContext, *attributeName, "{{interface_name}}", info.Holder(), info.GetIsolate());
+    if (!BindingSecurity::shouldAllowAccessToFrame(info.GetIsolate(), impl->frame(), exceptionState)) {
+        exceptionState.throwIfNeeded();
+        return;
+    }
+    {% endif %}
+    if (info.This()->IsObject())

[haraken, 2015/02/25 14:27:34]

Not related to your CL, I guess this is always true. I think we can change this
to ASSERT(info.This()->IsObject()) in a follow-up CL.

[Yuki, 2015/02/25 14:33:12]

Will do.

+        v8::Local<v8::Object>::Cast(info.This())->ForceSet(name, v8Value);
+}
+
+{% endif %}
+{% if has_constructor_attributes %}
+static void {{cpp_class}}ForceSetAttributeOnThisCallback(v8::Local<v8::String> name, v8::Local<v8::Value> v8Value, const v8::PropertyCallbackInfo<void>& info)

[haraken, 2015/02/25 14:27:34]

Not related to your CL, do we need this method indirection?

[Yuki, 2015/02/25 14:33:12]

I'm still studying what constructor attributes are, but they seem very close to
replaceable attributes.  (So I'm working on replaceable attributes before
constructor attributes.)

Maybe, this function will be gone when I work on constructor attributes.

+{
+    {{cpp_class_or_partial}}V8Internal::{{cpp_class}}ForceSetAttributeOnThis(name, v8Value, info);
+}
+
+{% endif %}
+{% endblock %}
+{##############################################################################}
 {% from 'attributes.cpp' import constructor_getter_callback,
        attribute_getter, attribute_getter_callback,
        attribute_setter, attribute_setter_callback,
        attribute_getter_implemented_in_private_script,
        attribute_setter_implemented_in_private_script
        with context %}
 {% for attribute in attributes if not attribute.constructor_type %}
 {% if attribute.should_be_exposed_to_script %}
 {% for world_suffix in attribute.world_suffixes %}
 {% if not attribute.has_custom_getter %}
 {{attribute_getter(attribute, world_suffix)}}
 {% endif %}
 {{attribute_getter_callback(attribute, world_suffix)}}
-{% if not attribute.is_read_only or attribute.put_forwards %}
+{% if (not attribute.is_read_only or
+       attribute.put_forwards or
+       attribute.is_replaceable) %}
 {% if not attribute.has_custom_setter %}
 {{attribute_setter(attribute, world_suffix)}}
 {% endif %}
 {{attribute_setter_callback(attribute, world_suffix)}}
 {% endif %}
 {% endfor %}
 {% endif %}
 {% endfor %}
 {##############################################################################}
 {% block constructor_getter %}
@@ skipping 10 matching lines @@
 
 {% endif %}
 {% endblock %}
 {##############################################################################}
 {% for attribute in attributes if attribute.needs_constructor_getter_callback %}
 {% for world_suffix in attribute.world_suffixes %}
 {{constructor_getter_callback(attribute, world_suffix)}}
 {% endfor %}
 {% endfor %}
 {##############################################################################}
-{% block replaceable_attribute_setter_and_callback %}
-{% if has_replaceable_attributes or has_constructor_attributes %}
-static void {{cpp_class}}ForceSetAttributeOnThis(v8::Local<v8::String> name, v8::Local<v8::Value> v8Value, const v8::PropertyCallbackInfo<void>& info)
-{
-    {% if is_check_security %}
-    {{cpp_class}}* impl = {{v8_class}}::toImpl(info.Holder());
-    v8::String::Utf8Value attributeName(name);
-    ExceptionState exceptionState(ExceptionState::SetterContext, *attributeName, "{{interface_name}}", info.Holder(), info.GetIsolate());
-    if (!BindingSecurity::shouldAllowAccessToFrame(info.GetIsolate(), impl->frame(), exceptionState)) {
-        exceptionState.throwIfNeeded();
-        return;
-    }
-    {% endif %}
-    if (info.This()->IsObject())
-        v8::Local<v8::Object>::Cast(info.This())->ForceSet(name, v8Value);
-}
-
-static void {{cpp_class}}ForceSetAttributeOnThisCallback(v8::Local<v8::String> name, v8::Local<v8::Value> v8Value, const v8::PropertyCallbackInfo<void>& info)
-{
-    {{cpp_class_or_partial}}V8Internal::{{cpp_class}}ForceSetAttributeOnThis(name, v8Value, info);
-}
-
-{% endif %}
-{% endblock %}
-{##############################################################################}
 {% block security_check_functions %}
 {% if has_access_check_callbacks %}
 bool indexedSecurityCheck(v8::Local<v8::Object> host, uint32_t index, v8::AccessType type, v8::Local<v8::Value>)
 {
     {{cpp_class}}* impl = {{v8_class}}::toImpl(host);
     return BindingSecurity::shouldAllowAccessToFrame(v8::Isolate::GetCurrent(), impl->frame(), DoNotReportSecurityError);
 }
 
 bool namedSecurityCheck(v8::Local<v8::Object> host, v8::Local<v8::Value> key, v8::AccessType type, v8::Local<v8::Value>)
 {
@@ skipping 375 matching lines @@
 {% endfor %}
 {% for attribute in attributes if attribute.is_implemented_in_private_script %}
 {{attribute_getter_implemented_in_private_script(attribute)}}
 {% if not attribute.is_read_only or attribute.put_forwards %}
 {{attribute_setter_implemented_in_private_script(attribute)}}
 {% endif %}
 {% endfor %}
 {% block partial_interface %}{% endblock %}
 } // namespace blink
 {% endfilter %}