Simplify V8 operations in renderer ipc code.
authorCheng Zhao <zcbenz@gmail.com>
Thu, 5 Dec 2013 14:12:27 +0000 (22:12 +0800)
committerCheng Zhao <zcbenz@gmail.com>
Thu, 5 Dec 2013 14:12:27 +0000 (22:12 +0800)
renderer/api/atom_api_renderer_ipc.cc

index d126347..ddf16a5 100644 (file)
@@ -45,10 +45,10 @@ RenderView* GetCurrentRenderView() {
 v8::Handle<v8::Value> RendererIPC::Send(const v8::Arguments &args) {
   v8::HandleScope scope;
 
-  if (!args[0]->IsString())
+  string16 channel;
+  if (!FromV8Arguments(args, &channel))
     return node::ThrowTypeError("Bad argument");
 
-  string16 channel = FromV8Value(args[0]);
   RenderView* render_view = GetCurrentRenderView();
 
   // Convert Arguments to Array, so we can use V8ValueConverter to convert it
@@ -78,12 +78,10 @@ v8::Handle<v8::Value> RendererIPC::Send(const v8::Arguments &args) {
 v8::Handle<v8::Value> RendererIPC::SendSync(const v8::Arguments &args) {
   v8::HandleScope scope;
 
-  if (!args[0]->IsString())
+  string16 channel;
+  if (!FromV8Arguments(args, &channel))
     return node::ThrowTypeError("Bad argument");
 
-  v8::Handle<v8::Context> context = v8::Context::GetCurrent();
-  string16 channel = FromV8Value(args[0]);
-
   // Convert Arguments to Array, so we can use V8ValueConverter to convert it
   // to ListValue.
   v8::Local<v8::Array> v8_args = v8::Array::New(args.Length() - 1);
@@ -91,7 +89,8 @@ v8::Handle<v8::Value> RendererIPC::SendSync(const v8::Arguments &args) {
     v8_args->Set(i, args[i + 1]);
 
   scoped_ptr<V8ValueConverter> converter(V8ValueConverter::create());
-  scoped_ptr<base::Value> arguments(converter->FromV8Value(v8_args, context));
+  scoped_ptr<base::Value> arguments(
+      converter->FromV8Value(v8_args, v8::Context::GetCurrent()));
 
   DCHECK(arguments && arguments->IsType(base::Value::TYPE_LIST));