IDL compiler: [SetReference]

Source/bindings/templates/interface.cpp

 {% extends 'interface_base.cpp' %}
 
 
 {##############################################################################}
 {% macro attribute_configuration(attribute) %}
 {% set getter_callback =
        '%sV8Internal::%sAttributeGetterCallback' %
             (cpp_class, attribute.name)
        if not attribute.constructor_type else
        '{0}V8Internal::{0}ConstructorGetter'.format(interface_name) %}
@@ skipping 240 matching lines @@
     V8DOMWrapper::associateObjectWithWrapper<{{v8_class}}>(event.release(), &{{v8_class}}::wrapperTypeInfo, wrapper, info.GetIsolate(), WrapperConfiguration::Dependent);
     v8SetReturnValue(info, wrapper);
 }
 
 {% endif %}
 {% endblock %}
 
 
 {##############################################################################}
 {% block visit_dom_wrapper %}
-{% if generate_visit_dom_wrapper_function %}
+{% if check_reachable_method or set_reference_list %}
 void {{v8_class}}::visitDOMWrapper(void* object, const v8::Persistent<v8::Object>& wrapper, v8::Isolate* isolate)
 {
     {{cpp_class}}* impl = fromInternalPointer(object);
-    if (Node* owner = impl->{{generate_visit_dom_wrapper_function}}()) {
+    {% if set_reference_list %}
+    v8::Local<v8::Object> creationContext = v8::Local<v8::Object>::New(isolate, wrapper);

[haraken, 2013/12/17 09:13:09]

kouhei: Why do we need to create a new Local handle here? Can't we simply use
|wrapper| as |creationContext| ?

[kouhei (in TOK), 2013/12/17 09:15:00]

I don't remember if there was a specific reason. I'll try your suggestion later.

+    V8WrapperInstantiationScope scope(creationContext, isolate);
+    {% for set_reference in set_reference_list %}
+    {{set_reference.idl_type}}* {{set_reference.name}} = impl->{{set_reference.name}}();
+    if ({{set_reference.name}}) {
+        if (!DOMDataStore::containsWrapper<{{set_reference.v8_type}}>({{set_reference.name}}, isolate))
+            wrap({{set_reference.name}}, creationContext, isolate);
+        DOMDataStore::setWrapperReference<{{set_reference.v8_type}}>(wrapper, {{set_reference.name}}, isolate);
+    }
+    {% endfor %}
+    {% endif %}
+    {% if check_reachable_method %}
+    if (Node* owner = impl->{{check_reachable_method}}()) {

[haraken, 2013/12/17 09:13:09]

check_reachable_method => reachable_node_function ?

(I think you're using "function" for C++ side function names.)

[Nils Barth (inactive), 2013/12/18 01:57:49]

Good point, fixed!

         setObjectGroup(V8GCController::opaqueRootForGC(owner, isolate), wrapper, isolate);
         return;
     }
+    {% endif %}
     setObjectGroup(object, wrapper, isolate);
 }
 
 {% endif %}
 {% endblock %}
 
 
 {##############################################################################}
 {% block class_attributes %}
 {# FIXME: rename to install_attributes and put into configure_class_template #}
@@ skipping 446 matching lines @@
     fromInternalPointer(object)->deref();
 }
 
 template<>
 v8::Handle<v8::Value> toV8NoInline({{cpp_class}}* impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
 {
     return toV8(impl, creationContext, isolate);
 }
 
 {% endblock %}