[Android] Implement renderer side of Gin Java Bridge

content/renderer/java/gin_java_bridge_object.cc

Index: content/renderer/java/gin_java_bridge_object.cc
diff --git a/content/renderer/java/gin_java_bridge_object.cc b/content/renderer/java/gin_java_bridge_object.cc
new file mode 100644
index 0000000000000000000000000000000000000000..b4a64e33a648cf3099fa01315f312c17c9f06d8a
--- /dev/null
+++ b/content/renderer/java/gin_java_bridge_object.cc
@@ -0,0 +1,175 @@
+// Copyright 2014 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 "content/renderer/java/gin_java_bridge_object.h"
+
+#include "base/strings/utf_string_conversions.h"
+#include "content/common/android/gin_java_bridge_value.h"
+#include "content/public/renderer/v8_value_converter.h"
+#include "content/renderer/java/gin_java_bridge_value_converter.h"
+#include "gin/function_template.h"
+#include "third_party/WebKit/public/web/WebFrame.h"
+#include "third_party/WebKit/public/web/WebKit.h"
+
+namespace content {
+
+namespace {
+
+const char kMethodInvocationErrorMessage[] =
+    "Java bridge method invocation error";
+
+}  // namespace
+
+
+// static
+GinJavaBridgeObject* GinJavaBridgeObject::Inject(
+    blink::WebFrame* frame,
+    const base::WeakPtr<GinJavaBridgeDispatcher>& dispatcher,
+    const std::string& object_name,
+    GinJavaBridgeDispatcher::ObjectID object_id) {
+  GinJavaBridgeObject* result = new GinJavaBridgeObject(
+      blink::mainThreadIsolate(), dispatcher, object_id);
+  if (InjectExisting(frame, result, object_name)) {
+    return result;
+  } else {
+    delete result;
+    return NULL;
+  }
+}
+
+// static
+GinJavaBridgeObject* GinJavaBridgeObject::InjectAnonymous(
+    const base::WeakPtr<GinJavaBridgeDispatcher>& dispatcher,
+    GinJavaBridgeDispatcher::ObjectID object_id) {
+  return new GinJavaBridgeObject(
+      blink::mainThreadIsolate(), dispatcher, object_id);
+}
+
+// static
+bool GinJavaBridgeObject::InjectExisting(blink::WebFrame* frame,
+                                         GinJavaBridgeObject* object,
+                                         const std::string& object_name) {
+  v8::Isolate* isolate = blink::mainThreadIsolate();
+  v8::HandleScope handle_scope(isolate);
+  v8::Handle<v8::Context> context = frame->mainWorldScriptContext();
+  if (context.IsEmpty())
+    return false;
+
+  v8::Context::Scope context_scope(context);
+  v8::Handle<v8::Object> global = context->Global();
+  gin::Handle<GinJavaBridgeObject> controller =
+      gin::CreateHandle(isolate, object);

[jochen (gone - plz use gerrit), 2014/05/05 10:55:49]

CreateHandle will delete object if it fails to create controller... so if you
return in line 64, you'll have a double-free in line 36

[mnaganov (inactive), 2014/05/06 10:54:15]

Thanks for pointing this out! I missed that fact that WrappableBase deletes
itself in case of wrapper creation failure.

+  if (controller.IsEmpty())
+    return false;
+
+  global->Set(gin::StringToV8(isolate, object_name), controller.ToV8());
+  return true;
+}
+
+GinJavaBridgeObject::GinJavaBridgeObject(
+    v8::Isolate* isolate,
+    const base::WeakPtr<GinJavaBridgeDispatcher>& dispatcher,
+    GinJavaBridgeDispatcher::ObjectID object_id)
+    : gin::NamedPropertyInterceptor(isolate, this),
+      dispatcher_(dispatcher),
+      object_id_(object_id),
+      converter_(new GinJavaBridgeValueConverter()) {
+}
+
+GinJavaBridgeObject::~GinJavaBridgeObject() {
+  if (dispatcher_)
+    dispatcher_->OnGinJavaBridgeObjectDeleted(object_id_);
+}
+
+gin::ObjectTemplateBuilder GinJavaBridgeObject::GetObjectTemplateBuilder(
+    v8::Isolate* isolate) {
+  return gin::Wrappable<GinJavaBridgeObject>::GetObjectTemplateBuilder(isolate)
+      .AddNamedPropertyInterceptor();
+}
+
+v8::Local<v8::Value> GinJavaBridgeObject::GetNamedProperty(
+    v8::Isolate* isolate,
+    const std::string& property) {
+  if (dispatcher_ && dispatcher_->HasJavaMethod(object_id_, property)) {
+    return gin::CreateFunctionTemplate(
+               isolate,
+               base::Bind(&GinJavaBridgeObject::InvokeMethod,
+                          base::Unretained(this),
+                          property))->GetFunction();
+  } else {
+    return v8::Local<v8::Value>();
+  }
+}
+
+std::vector<std::string> GinJavaBridgeObject::EnumerateNamedProperties(
+    v8::Isolate* isolate) {
+  std::set<std::string> method_names;
+  if (dispatcher_) {
+    dispatcher_->GetJavaMethods(object_id_, &method_names);

[jochen (gone - plz use gerrit), 2014/05/05 10:55:49]

nit. no { }

[mnaganov (inactive), 2014/05/06 10:54:15]

Done.

+  }
+  return std::vector<std::string> (method_names.begin(), method_names.end());
+}
+
+v8::Handle<v8::Value> GinJavaBridgeObject::InvokeMethod(
+    const std::string& name,
+    gin::Arguments* args) {
+  if (!dispatcher_) {
+    args->isolate()->ThrowException(v8::Exception::Error(gin::StringToV8(
+        args->isolate(), kMethodInvocationErrorMessage)));
+    return v8::Undefined(args->isolate());
+  }
+
+  base::ListValue arguments;
+  {
+    v8::HandleScope handle_scope(args->isolate());
+    v8::Handle<v8::Context> context = args->isolate()->GetCurrentContext();
+    v8::Handle<v8::Value> val;
+    while (args->GetNext(&val)) {
+      scoped_ptr<base::Value> arg(converter_->FromV8Value(val, context));
+      if (arg.get()) {
+        arguments.Append(arg.release());
+      } else {
+        arguments.Append(base::Value::CreateNullValue());
+      }
+    }
+  }
+
+  scoped_ptr<base::Value> result =
+      dispatcher_->InvokeJavaMethod(object_id_, name, arguments);
+  if (!result.get()) {
+    args->isolate()->ThrowException(v8::Exception::Error(gin::StringToV8(
+        args->isolate(), kMethodInvocationErrorMessage)));
+    return v8::Undefined(args->isolate());
+  }
+  if (!result->IsType(base::Value::TYPE_BINARY)) {
+    return converter_->ToV8Value(result.get(),
+                                 args->isolate()->GetCurrentContext());
+  }
+
+  scoped_ptr<const GinJavaBridgeValue> gin_value =
+      GinJavaBridgeValue::FromValue(result.get());
+  if (gin_value->IsType(GinJavaBridgeValue::TYPE_OBJECT_ID)) {
+    GinJavaBridgeObject* result = NULL;
+    GinJavaBridgeDispatcher::ObjectID object_id;
+    if (gin_value->GetAsObjectID(&object_id)) {
+      result = dispatcher_->GetObject(object_id);
+    }
+    if (result) {
+      gin::Handle<GinJavaBridgeObject> controller =
+          gin::CreateHandle(args->isolate(), result);
+      if (controller.IsEmpty())
+        return v8::Undefined(args->isolate());
+      return controller.ToV8();
+    }
+  } else if (gin_value->IsType(GinJavaBridgeValue::TYPE_NONFINITE)) {
+    float float_value;
+    gin_value->GetAsNonFinite(&float_value);
+    return v8::Number::New(args->isolate(), float_value);
+  }
+  return v8::Undefined(args->isolate());
+}
+
+gin::WrapperInfo GinJavaBridgeObject::kWrapperInfo = {gin::kEmbedderNativeGin};
+
+}  // namespace content