Implement gin::ObjectTemplateBuilder

mojo/apps/js/threading.cc

Index: mojo/apps/js/threading.cc
diff --git a/mojo/apps/js/threading.cc b/mojo/apps/js/threading.cc
index 09536cd25b1be981d15bf01037f38e623db43c0c..23ffb132ab06ea85cbbe3e7f8511aafc8eb942cf 100644
--- a/mojo/apps/js/threading.cc
+++ b/mojo/apps/js/threading.cc
@@ -7,6 +7,7 @@
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "gin/function_template.h"
+#include "gin/object_template_builder.h"
 #include "gin/per_isolate_data.h"
 #include "mojo/public/bindings/js/handle.h"
 
@@ -31,9 +32,14 @@ v8::Local<v8::ObjectTemplate> Threading::GetTemplate(v8::Isolate* isolate) {
       &g_wrapper_info);
 
   if (templ.IsEmpty()) {
-    templ = v8::ObjectTemplate::New();
-    templ->Set(gin::StringToSymbol(isolate, "quit"),
-               gin::CreateFunctionTemplate(isolate, base::Bind(Quit)));
+    // Don't forget to call SetInitialHandle before getting the template.
+    DCHECK(g_initial_handle != MOJO_HANDLE_INVALID);

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

This is gone now.  You can delete these lines.

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

Done.

+
+    gin::ObjectTemplateBuilder builder(isolate);
+    templ = builder
+        .SetMethod("quit", Quit)
+        .Build();
+
     data->SetObjectTemplate(&g_wrapper_info, templ);
   }