Simplify context setup, set properties on the global object directly instead of using templates

src/GlobalJsObject.cpp

Index: src/GlobalJsObject.cpp
===================================================================
--- a/src/GlobalJsObject.cpp
+++ b/src/GlobalJsObject.cpp
@@ -63,29 +63,19 @@ namespace
 
     // We should actually return the timer ID here, which could be
     // used via clearTimeout(). But since we don't seem to need
     // clearTimeout(), we can save that for later.
     return v8::Undefined();
   }
 }
 
-v8::Handle<v8::ObjectTemplate> GlobalJsObject::Create(
-  JsEngine& jsEngine)
+JsValuePtr GlobalJsObject::Setup(JsEngine& jsEngine, JsValuePtr obj)
 {
-  const v8::Locker locker(v8::Isolate::GetCurrent());
-  v8::HandleScope handleScope;
-  const v8::Handle<v8::ObjectTemplate> global = v8::ObjectTemplate::New();
-  const v8::Handle<v8::FunctionTemplate> setTimeoutFunction =
-    v8::FunctionTemplate::New(SetTimeoutCallback,
-                              v8::External::New(&jsEngine));
-  global->Set(v8::String::New("setTimeout"), setTimeoutFunction);
-  const v8::Handle<v8::ObjectTemplate> fileSystemObject =
-    FileSystemJsObject::Create(jsEngine);
-  global->Set(v8::String::New("_fileSystem"), fileSystemObject);
-  const v8::Handle<v8::ObjectTemplate> webRequestObject =
-    WebRequestJsObject::Create(jsEngine);
-  global->Set(v8::String::New("_webRequest"), webRequestObject);
-  const v8::Handle<v8::ObjectTemplate> consoleObject =
-    ConsoleJsObject::Create(jsEngine);
-  global->Set(v8::String::New("console"), consoleObject);
-  return handleScope.Close(global);
+  obj->SetProperty("setTimeout", jsEngine.NewCallback(::SetTimeoutCallback));
+  obj->SetProperty("_fileSystem",
+      FileSystemJsObject::Setup(jsEngine, jsEngine.NewObject()));
+  obj->SetProperty("_webRequest",
+      WebRequestJsObject::Setup(jsEngine, jsEngine.NewObject()));
+  obj->SetProperty("console",
+      ConsoleJsObject::Setup(jsEngine, jsEngine.NewObject()));
+  return obj;
 }