IDL: ArrayBuffer and ArrayBufferView support for union types

Source/bindings/templates/union.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"
 #include "{{header_filename}}"
 
+{% macro install_hasinstance_check(member) %}

[haraken, 2014/11/04 02:20:14]

"install_" sounds a bit strange. hasinstance_check or
to_impl_with_hasinstance_check ?

We normally use "install" to install properties onto V8DOMConfiguration.

[bashi, 2014/11/04 02:30:32]

Done.

[Yuki, 2014/11/04 06:35:14]

We already have to_impl_with_type_check block in interface.cpp, and it seems a
little confusing.

Plus, the essential part of the macro is not purely toImpl.  The macro is
essentially:
    impl.setType(V8Type::toImpl(v8Value));
which is essentially:
    impl = V8Type::toImpl(v8Value);
which includes an assignment in addition to toImpl.

So I'd prefer another name which represents/implies an assignment.  If 'install'
is not good, can we use 'set', 'assign' or something else?

Also I'd recommend to specify arguments for the input and output.
  {% macro foo(impl, v8Value, member) %}
  impl.setType(V8Type::toImpl(v8Value));
  {% endmacro %}

  {# A caller knows the actual input and output. #}
  {{ foo('impl', 'v8Value', member) }}
It makes scope of variables clearer.

[Jens Widell, 2014/11/04 07:22:56]

Oh, naming of things is always fun! :-)

How about "assign_and_return_if_hasinstance"?

[bashi, 2014/11/04 08:35:05]

"assign_and_return_if_hasinstnace" sounds good:) Done.

+if (V8{{member.type_name}}::hasInstance(v8Value, isolate)) {
+    {{member.cpp_local_type}} cppValue = V8{{member.type_name}}::toImpl(v8::Handle<v8::Object>::Cast(v8Value));
+    impl.set{{member.type_name}}(cppValue);
+    return;
+}
+{% endmacro %}
 {% for filename in cpp_includes %}
 #include "{{filename}}"
 {% endfor %}
 
 namespace blink {
 
 {% for container in containers %}
 {{container.cpp_class}}::{{container.cpp_class}}()
     : m_type(SpecificTypeNone)
 {
@@ skipping 29 matching lines @@
        moment. Should be fixed once we implement all necessary conversion steps
        below. #}
     if (v8Value.IsEmpty())
         return;
 
     {# The numbers in the following comments refer to the steps described in
        http://heycam.github.io/webidl/#es-union
        FIXME: Implement all necessary steps #}
     {# 3. Platform objects (interfaces) #}
     {% for interface in container.interface_types %}
-    if (V8{{interface.type_name}}::hasInstance(v8Value, isolate)) {
-        {{interface.cpp_local_type}} cppValue = V8{{interface.type_name}}::toImpl(v8::Handle<v8::Object>::Cast(v8Value));
-        impl.set{{interface.type_name}}(cppValue);
-        return;
-    }
+    {{install_hasinstance_check(interface) | indent}}
 
     {% endfor %}
+    {# 8. ArrayBuffer #}
+    {% if container.array_buffer_type %}
+    {{install_hasinstance_check(container.array_buffer_type) | indent}}
+
+    {% endif %}
+    {# 9., 10. ArrayBufferView #}
+    {# FIXME: Individual typed arrays (e.g. Uint8Array) aren't supported yet. #}
+    {% if container.array_buffer_view_type %}
+    {{install_hasinstance_check(container.array_buffer_view_type) | indent}}
+
+    {% endif %}
     {% if container.dictionary_type %}
     {# 12. Dictionaries #}
     {# FIXME: This should also check "object but not Date or RegExp". Add checks
        when we implement conversions for Date and RegExp. #}
     if (isUndefinedOrNull(v8Value) || v8Value->IsObject()) {
         {{container.dictionary_type.cpp_local_type}} cppValue = V8{{container.dictionary_type.type_name}}::toImpl(isolate, v8Value, exceptionState);
         if (!exceptionState.hadException())
             impl.set{{container.dictionary_type.type_name}}(cppValue);
         return;
     }
@@ skipping 56 matching lines @@
     if (impl.is{{member.type_name}}())
         return {{member.cpp_value_to_v8_value}};
 
     {% endfor %}
     ASSERT_NOT_REACHED();
     return v8::Handle<v8::Value>();
 }
 
 {% endfor %}
 } // namespace blink