Eagerly install Origin Trial features on window

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

 {% filter format_blink_cpp_source_code %}
 
 {% include 'copyright_block.txt' %}
 #include "{{v8_class_or_partial}}.h"
 
 {% for filename in cpp_includes if filename != '%s.h' % cpp_class_or_partial %}
 #include "{{filename}}"
 {% endfor %}
 
 namespace blink {
@@ skipping 372 matching lines @@
 }
 
 {% endif %}{# not is_array_buffer_or_view #}
 {% endblock %}
 {##############################################################################}
 {% block origin_trials %}
 {% from 'attributes.cpp.tmpl' import attribute_configuration with context %}
 {% from 'constants.cpp.tmpl' import constant_configuration with context %}
 {% from 'methods.cpp.tmpl' import method_configuration with context %}
 {% for origin_trial_feature in origin_trial_features %}{{newline}}
-void {{v8_class_or_partial}}::install{{origin_trial_feature.name}}(v8::Isolate* isolate, const DOMWrapperWorld& world, v8::Local<v8::Object> instance, v8::Local<v8::Object> prototype, v8::Local<v8::Function> interface) {
+void {{v8_class_or_partial}}::install{{origin_trial_feature.name}}(const ScriptState* scriptState, v8::Local<v8::Object> instance, v8::Local<v8::Object> prototype, v8::Local<v8::Function> interface) {

[Yuki, 2016/11/01 11:37:54]

I have the same thought as I commented at installAttributeInternal in
V8DOMConfiguration.cpp.

|instance| is not an instance of type X, right?

I'd like to have a comment to elaborate what each argument is.

I welcome anythings that make things clearer.

Probably you'd need |instance|, |prototype|, |interface|, and
|interface_exposure_target| (I don't come up with a good name) as arguments?


Just in case, why other functions take instance, prototype, and interface is:
a) [Unforgeable] attributes/operations must be placed on |instance|, and
b) other attributes/operations must be placed on |prototype|, and
c) constants must be placed on |interface| and optionally on |prototype|.
but,
d) the interface objects should be exposed on the global object in general.
I guess you'd need all of them.

[iclelland, 2016/11/01 17:06:27]

It *is*, though. The method takes |instance|, |prototype|, and |interface|, and
they correspond to the targets for V8DOMConfiguration::OnInstance, OnPrototype
and OnInterface. In almost every case, not all three of these will be used, but
it was easiest for the template to generate methods taking all three.
|instance| and |interface_exposure_target| are the same object, I believe.
That matches my understanding.

We haven't tried to handle [Unforgeable] in origin trials yet, and I suspect it
wouldn't work very well. b) and c) are the reasons that we have the |prototype|
and |interface| arguments.
This case is handled by having the interface objects configured as OnInstance
attributes of the globals, so in those cases, the |instance| argument is used,
and should be the global object. In other cases, |instance| is unused, and so a
dummy v8::Object is passed instead.

+  v8::Isolate* isolate = scriptState->isolate();
+  const DOMWrapperWorld& world = scriptState->world();
   {% if attributes | for_origin_trial_feature(origin_trial_feature.name) or
         methods | method_for_origin_trial_feature(origin_trial_feature.name, is_partial) %}
   v8::Local<v8::FunctionTemplate> interfaceTemplate = {{v8_class}}::wrapperTypeInfo.domTemplate(isolate, world);
   v8::Local<v8::Signature> signature = v8::Signature::New(isolate, interfaceTemplate);
   ALLOW_UNUSED_LOCAL(signature);
   {% endif %}
   {# Origin-Trial-enabled attributes #}
   {% for attribute in attributes | for_origin_trial_feature(origin_trial_feature.name) | unique_by('name') | sort %}
   {% if attribute.is_data_type_property %}
   const V8DOMConfiguration::AttributeConfiguration attribute{{attribute.name}}Configuration = {{attribute_configuration(attribute)}};
-  V8DOMConfiguration::installAttribute(isolate, world, instance, prototype, attribute{{attribute.name}}Configuration);
+  V8DOMConfiguration::installAttribute(scriptState, instance, prototype, attribute{{attribute.name}}Configuration);
   {% else %}
   const V8DOMConfiguration::AccessorConfiguration accessor{{attribute.name}}Configuration = {{attribute_configuration(attribute)}};
   V8DOMConfiguration::installAccessor(isolate, world, instance, prototype, interface, signature, accessor{{attribute.name}}Configuration);
   {% endif %}
   {% endfor %}
   {# Origin-Trial-enabled constants #}
   {% for constant in constants | for_origin_trial_feature(origin_trial_feature.name) | unique_by('name') | sort %}
   {% set constant_name = constant.name.title().replace('_', '') %}
   const V8DOMConfiguration::ConstantConfiguration constant{{constant_name}}Configuration = {{constant_configuration(constant)}};
   V8DOMConfiguration::installConstant(isolate, interface, prototype, constant{{constant_name}}Configuration);
   {% endfor %}
   {# Origin-Trial-enabled methods (no overloads) #}
   {% for method in methods | method_for_origin_trial_feature(origin_trial_feature.name, is_partial) | unique_by('name') | sort %}
   {% set method_name = method.name.title().replace('_', '') %}
   const V8DOMConfiguration::MethodConfiguration method{{method_name}}Configuration = {{method_configuration(method)}};
   V8DOMConfiguration::installMethod(isolate, world, instance, prototype, interface, signature, method{{method_name}}Configuration);
   {% endfor %}
 }
 
-void {{v8_class_or_partial}}::install{{origin_trial_feature.name}}(ScriptState* scriptState, v8::Local<v8::Object> instance) {
+void {{v8_class_or_partial}}::install{{origin_trial_feature.name}}(const ScriptState* scriptState, v8::Local<v8::Object> instance) {
   V8PerContextData* perContextData = V8PerContextData::from(scriptState->context());
   v8::Local<v8::Object> prototype = perContextData->prototypeForType(&{{v8_class}}::wrapperTypeInfo);
   v8::Local<v8::Function> interface = perContextData->constructorForType(&{{v8_class}}::wrapperTypeInfo);
   ALLOW_UNUSED_LOCAL(interface);
-  install{{origin_trial_feature.name}}(scriptState->isolate(), scriptState->world(), instance, prototype, interface);
+  install{{origin_trial_feature.name}}(scriptState, instance, prototype, interface);
 }
 {% if not origin_trial_feature.needs_instance %}
 
-void {{v8_class_or_partial}}::install{{origin_trial_feature.name}}(ScriptState* scriptState) {
+void {{v8_class_or_partial}}::install{{origin_trial_feature.name}}(const ScriptState* scriptState) {
   install{{origin_trial_feature.name}}(scriptState, v8::Local<v8::Object>());
 }
 {% endif %}
 {% endfor %}
 {% endblock %}
 {##############################################################################}
 {% block get_dom_template %}{% endblock %}
 {% block get_dom_template_for_named_properties_object %}{% endblock %}
 {% block has_instance %}{% endblock %}
 {% block to_impl %}{% endblock %}
 {% block to_impl_with_type_check %}{% endblock %}
 {##############################################################################}
 {% block prepare_prototype_and_interface_object %}{% endblock %}
 {##############################################################################}
 {% for method in methods if method.is_implemented_in_private_script and method.visible %}
 {{method_implemented_in_private_script(method)}}
 {% endfor %}
 {% for attribute in attributes if attribute.is_implemented_in_private_script %}
 {{attribute_getter_implemented_in_private_script(attribute)}}
 {% if attribute.has_setter %}
 {{attribute_setter_implemented_in_private_script(attribute)}}
 {% endif %}
 {% endfor %}
 {% block partial_interface %}{% endblock %}
 }  // namespace blink
 
 {% endfilter %}{# format_blink_cpp_source_code #}