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

Source/bindings/tests/results/core/V8SVGTestInterface.cpp

Index: Source/bindings/tests/results/core/V8SVGTestInterface.cpp
diff --git a/Source/bindings/tests/results/core/V8SVGTestInterface.cpp b/Source/bindings/tests/results/core/V8SVGTestInterface.cpp
index 62a65ec8ef3ecbaec426d1903abdf7617cc0f173..cda353fd9a60ada79543f74e277fef8e41e3e51f 100644
--- a/Source/bindings/tests/results/core/V8SVGTestInterface.cpp
+++ b/Source/bindings/tests/results/core/V8SVGTestInterface.cpp
@@ -102,9 +102,10 @@ v8::Handle<v8::Object> V8SVGTestInterface::findInstanceInPrototypeChain(v8::Hand
     return V8PerIsolateData::from(isolate)->findInstanceInPrototypeChain(&wrapperTypeInfo, v8Value);
 }
 
+

[haraken, 2014/10/14 15:11:18]

Unnecessary empty line.

[Yuki, 2014/10/15 09:35:23]

Will check when I add a run-binding-test.

 SVGTestInterface* V8SVGTestInterface::toImplWithTypeCheck(v8::Isolate* isolate, v8::Handle<v8::Value> value)
 {
-    return hasInstance(value, isolate) ? blink::toScriptWrappableBase(v8::Handle<v8::Object>::Cast(value))->toImpl<SVGTestInterface>() : 0;
+    return hasInstance(value, isolate) ? toImpl(v8::Handle<v8::Object>::Cast(value)) : 0;

[haraken, 2014/10/14 15:11:18]

Isn't this a regression? We want to use ScriptWrappable's virtual toImpl() as
much as possible.

[Yuki, 2014/10/15 09:35:23]

For v8::ArrayBuffer and its view, it's possible that the internal pointer is
null, and we cannot simply call internalPointer->toImpl<T>().

I'll revisit this point later.  Let me go on with this code for now.

 }