Fix promise resolution of FontFaceSet#load()

Source/bindings/v8/V8Binding.h

Index: Source/bindings/v8/V8Binding.h
diff --git a/Source/bindings/v8/V8Binding.h b/Source/bindings/v8/V8Binding.h
index 1d2d7ce85842cbe5b2719d6233caa6243784e2a2..3e1e3d556611ab4c9011aa7580e1e0cce070f057 100644
--- a/Source/bindings/v8/V8Binding.h
+++ b/Source/bindings/v8/V8Binding.h
@@ -292,6 +292,17 @@ v8::Handle<v8::Value> v8Array(const HeapVector<T, inlineCapacity>& iterator, v8:
     return result;
 }
 
+template<typename T, size_t inlineCapacity>
+v8::Handle<v8::Value> v8ArrayNoInline(const Vector<T, inlineCapacity>& iterator, v8::Isolate* isolate)

[haraken, 2014/03/19 08:31:58]

I don't fully understand why you call this v8Array"NoInline". It's inlined.

Doesn't normal v8Array() work for you?

[Kunihiko Sakamoto, 2014/03/19 09:12:17]

Because this uses toV8NoInline(T) instead of toV8(T).
This function itself is inlined indeed ^^; any suggestions for naming?

Normal v8Array doesn't work. This code is used from FontFaceSet.cpp, and
toV8(FontFace) is defined in generated V8FontFace.h. We don't want to include
generated headers from Source/core. (see discussions in crbug.com/321569)

+{
+    v8::Local<v8::Array> result = v8::Array::New(isolate, iterator.size());
+    int index = 0;
+    typename Vector<T, inlineCapacity>::const_iterator end = iterator.end();
+    for (typename Vector<T, inlineCapacity>::const_iterator iter = iterator.begin(); iter != end; ++iter)
+        result->Set(v8::Integer::New(isolate, index++), toV8NoInline(WTF::getPtr(*iter), v8::Handle<v8::Object>(), isolate));
+    return result;
+}
+
 // Conversion flags, used in toIntXX/toUIntXX.
 enum IntegerConversionConfiguration {
     NormalConversion,