[WIP] IDL: Add limited serializer support and use for RTCIceCandidate

Source/bindings/core/v8/V8ObjectBuilder.cpp

+// Copyright 2015 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.
+
+#include "config.h"
+#include "bindings/core/v8/V8ObjectBuilder.h"
+
+#include "bindings/core/v8/V8Binding.h"
+
+namespace blink {
+
+V8ObjectBuilder::V8ObjectBuilder(v8::Isolate* isolate)
+    : m_isolate(isolate)
+    , m_object(v8::Object::New(isolate))
+{
+}
+
+ScriptValue V8ObjectBuilder::scriptValue() const
+{
+    return ScriptValue(ScriptState::current(m_isolate), m_object);
+}
+
+void V8ObjectBuilder::add(String name, const ScriptValue& value)
+{
+    add(name, value.v8Value());
+}
+
+void V8ObjectBuilder::addNull(String name)
+{
+    add(name, v8::Null(m_isolate));
+}
+
+void V8ObjectBuilder::addBoolean(String name, bool value)
+{
+    add(name, value ? v8::True(m_isolate) : v8::False(m_isolate));
+}
+
+void V8ObjectBuilder::addNumber(String name, double value)
+{
+    add(name, v8::Number::New(m_isolate, value));
+}
+
+void V8ObjectBuilder::addString(String name, String value)
+{
+    add(name, v8String(m_isolate, value));
+}
+
+V8ObjectBuilder V8ObjectBuilder::addObject(String name)

[bashi, 2015/03/24 00:49:04]

This looks different from other add methds() so addObject sounds a bit strange
to me.

[Jens Widell, 2015/03/24 07:10:03]

Yes, it's a bit unusual. Would probably be better to make the v8::Value add()
version public and do

  builder1.add("name", builder2.v8Value());

when building and adding a sub-object. Or add a 

  void add(String, const V8ObjectBuilder&);

overload, for that matter.

+{
+    V8ObjectBuilder builder(m_isolate);
+    add(name, builder.v8Value());
+    return builder;
+}
+
+void V8ObjectBuilder::add(String name, v8::Local<v8::Value> value)
+{
+    m_object->Set(m_isolate->GetCurrentContext(), v8String(m_isolate, name), value);
+}
+
+} // namespace blink