bindings: Adds DOMArrayBuffer, etc. as thin wrappers for ArrayBuffer, etc.

Source/bindings/scripts/v8_interface.py

Index: Source/bindings/scripts/v8_interface.py
diff --git a/Source/bindings/scripts/v8_interface.py b/Source/bindings/scripts/v8_interface.py
index 9cb32a59af27c5984346bf23bf7c82a820cc6830..b0d6d70bd13631412f902e8de9e043f5c38f2419 100644
--- a/Source/bindings/scripts/v8_interface.py
+++ b/Source/bindings/scripts/v8_interface.py
@@ -82,6 +82,23 @@ def interface_context(interface):
         header_includes.update(v8_types.includes_for_interface(parent_interface))
     extended_attributes = interface.extended_attributes
 
+    is_array_type = IdlType(interface.name).is_array_buffer_or_view

[haraken, 2014/10/16 05:41:41]

is_array_type => is_array_buffer_or_view (or simply is_array_buffer is fine)

I'd like to avoid array_type since it's confusing with a normal array type.
array_buffer is clearer.

[Yuki, 2014/10/16 14:21:50]

Done.

+    is_typed_array_type = IdlType(interface.name).is_typed_array
+    if is_array_type:
+        includes.add('bindings/core/v8/V8ArrayBuffer.h')
+    if interface.name == 'ArrayBufferView':
+        includes.update((
+            'bindings/core/v8/V8Int8Array.h',
+            'bindings/core/v8/V8Int16Array.h',
+            'bindings/core/v8/V8Int32Array.h',
+            'bindings/core/v8/V8Uint8Array.h',
+            'bindings/core/v8/V8Uint8ClampedArray.h',
+            'bindings/core/v8/V8Uint16Array.h',
+            'bindings/core/v8/V8Uint32Array.h',
+            'bindings/core/v8/V8Float32Array.h',
+            'bindings/core/v8/V8Float64Array.h',
+            'bindings/core/v8/V8DataView.h'))
+
     # [ActiveDOMObject]
     is_active_dom_object = 'ActiveDOMObject' in extended_attributes
 
@@ -157,12 +174,15 @@ def interface_context(interface):
         'header_includes': header_includes,
         'interface_name': interface.name,
         'is_active_dom_object': is_active_dom_object,
+        'is_array_type': is_array_type,

[haraken, 2014/10/16 05:41:41]

is_array_type => is_array_buffer

[Yuki, 2014/10/16 14:21:50]

Done.

         'is_check_security': is_check_security,
         'is_dependent_lifetime': is_dependent_lifetime,
         'is_event_target': inherits_interface(interface.name, 'EventTarget'),
         'is_exception': interface.is_exception,
         'is_node': inherits_interface(interface.name, 'Node'),
         'is_script_wrappable': is_script_wrappable,
+        'is_typed_array_type': is_typed_array_type,
+        'is_v8object': not is_array_type,  # The object is wrapped with v8::Object.

[haraken, 2014/10/16 05:41:41]

Hmm, I'd prefer not introducing is_v8object, given that the current code base
uses only v8::Object and v8::ArrayBuffer and that we don't want to introduce
unnecessary complexity to the code base.

Basically, everything should be wrapped with v8::Object. Typed array is a
special case for performance and it's unlikely that we'll add more special
casings.

[Yuki, 2014/10/16 14:21:50]

Done.

         'iterator_method': iterator_method,
         'lifetime': 'Dependent'
             if (has_visit_dom_wrapper or