Implement gin::ObjectTemplateBuilder

gin/test/gtest.cc

Index: gin/test/gtest.cc
diff --git a/gin/test/gtest.cc b/gin/test/gtest.cc
index a0cf70c75f5eacb4c7b7b152892e159b1a0f8d20..c869f6ea699b4678763c62f2c933c74e84cd4a1a 100644
--- a/gin/test/gtest.cc
+++ b/gin/test/gtest.cc
@@ -9,6 +9,7 @@
 #include "gin/arguments.h"
 #include "gin/converter.h"
 #include "gin/function_template.h"
+#include "gin/object_template_builder.h"
 #include "gin/per_isolate_data.h"
 #include "gin/public/wrapper_info.h"
 #include "gin/wrappable.h"
@@ -18,13 +19,7 @@ namespace gin {
 
 namespace {
 
-void Fail(const v8::FunctionCallbackInfo<v8::Value>& info) {
-  Arguments args(info);
-
-  std::string description;
-  if (!args.GetNext(&description))
-    return args.ThrowError();
-
+void Fail(const std::string& description) {
   FAIL() << description;
 }
 
@@ -53,15 +48,13 @@ v8::Local<v8::ObjectTemplate> GTest::GetTemplate(v8::Isolate* isolate) {
   v8::Local<v8::ObjectTemplate> templ =
       data->GetObjectTemplate(&g_wrapper_info);
   if (templ.IsEmpty()) {
-    templ = v8::ObjectTemplate::New();
-    templ->Set(StringToSymbol(isolate, "fail"),
-               v8::FunctionTemplate::New(Fail));
-    templ->Set(StringToSymbol(isolate, "expectTrue"),
-               CreateFunctionTemplate(isolate, base::Bind(ExpectTrue)));
-    templ->Set(StringToSymbol(isolate, "expectFalse"),
-               CreateFunctionTemplate(isolate, base::Bind(ExpectFalse)));
-    templ->Set(StringToSymbol(isolate, "expectEqual"),
-               CreateFunctionTemplate(isolate, base::Bind(ExpectEqual)));
+    ObjectTemplateBuilder builder(isolate);
+    templ = builder

[abarth-chromium, 2013/11/28 06:27:09]

We could even use a temporary and avoid giving the builder a name:

templ = ObjectTemplateBuilder(isolate)
             .SetMethod(...)
             .Build();

[Aaron Boodman, 2013/11/28 18:25:22]

Done.

+        .SetMethod("fail", Fail)
+        .SetMethod("expectTrue", ExpectTrue)
+        .SetMethod("expectFalse", ExpectFalse)
+        .SetMethod("expectEqual", ExpectEqual)
+        .Build();
     data->SetObjectTemplate(&g_wrapper_info, templ);
   }
   return templ;