[bindings] Remove custom binding usage in SQLResultSetRowList.

Source/bindings/core/v8/ToV8.h

Index: Source/bindings/core/v8/ToV8.h
diff --git a/Source/bindings/core/v8/ToV8.h b/Source/bindings/core/v8/ToV8.h
index e06dcd323a1ad8eccd8fff17325ed32c64cf9df7..0ef597a650d3517b2c627831fbe3c798f2dcbcaf 100644
--- a/Source/bindings/core/v8/ToV8.h
+++ b/Source/bindings/core/v8/ToV8.h
@@ -210,6 +210,15 @@ inline v8::Handle<v8::Value> toV8(const Vector<T, inlineCapacity>& value, v8::Ha
     return toV8SequenceInternal(value, creationContext, isolate);
 }
 
+template<typename T>
+inline v8::Handle<v8::Value> toV8(const Vector<std::pair<String, T>>& value, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)

[Yuki, 2015/03/04 14:11:04]

Since both of toV8(Vector<T>) and toV8(HeapVector<T>) returns v8::Array, and
both of them use toV8SequenceInternal, I think these two are closer to each
other than your new toV8(Vector<pair>).  So I'd prefer put this new function at
the end of the file (after HeapVector version).

+{
+    v8::Local<v8::Object> object = v8::Object::New(isolate);
+    for (unsigned i = 0; i < value.size(); ++i)
+        object->Set(v8String(isolate, value[i].first), value[i].second);

[Jens Widell, 2015/03/04 13:53:52]

By passing value[i].second directly to v8::Object::Set(), you are in fact
depending on T=v8::Handle<v8::Value>.

It would make more sense if the second argument to object->Set() was

  toV8(value[i].second, creationContext, isolate)

You could then call this with a Vector<std::pair<String, SQLValue>> argument
instead.

[vivekg, 2015/03/04 13:57:19]

Thanks Jens, I did the same thing initially. 
Though that would create a dependency on modules/v8/ToV8.h which needs to be
included in the core which I thought would be a layering violation. No? 

[Jens Widell, 2015/03/04 14:03:54]

It would certainly be wrong to include modules/v8/ToV8.h from core/v8/ToV8.h,
but luckily there would be no need for that. You would only need to have
included modules/v8/ToV8.h at the point where you call this toV8() function with
such an argument. (Which you already do.)

[Yuki, 2015/03/04 14:11:04]

Please use
  toV8(value[i].second, creationContext, isolate)
for the value so we can pass any v8-convertible types.
(Otherwise template<typename T> is meaningless.)

See toV8SequenceInternal() in this file for reference.

+    return object;
+}
+
 template<typename T, size_t inlineCapacity>
 inline v8::Handle<v8::Value> toV8(const HeapVector<T, inlineCapacity>& value, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
 {