Gin: Consolidate all the Converter<Wrappable> subclasses.

gin/wrappable_unittest.cc

 // Copyright 2013 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 "base/logging.h"
 #include "gin/arguments.h"
 #include "gin/handle.h"
 #include "gin/object_template_builder.h"
 #include "gin/per_isolate_data.h"
 #include "gin/public/isolate_holder.h"
@@ skipping 25 matching lines @@
 WrapperInfo MyObject::kWrapperInfo = { kEmbedderNativeGin };
 
 gin::Handle<MyObject> MyObject::Create(v8::Isolate* isolate) {
   return CreateHandle(isolate, new MyObject());
 }
 
 WrapperInfo* MyObject::GetWrapperInfo() {
   return &kWrapperInfo;
 }
 
-}  // namespace
 
-template<>
-struct Converter<MyObject*> : public WrappableConverter<MyObject> {};
+class MyObject2 : public Wrappable {
+ public:
+  MyObject2() {
+  }
+  static WrapperInfo kWrapperInfo;
+  virtual WrapperInfo* GetWrapperInfo() OVERRIDE;
+};
 
-namespace {
+WrapperInfo MyObject2::kWrapperInfo = { kEmbedderNativeGin };
 
-void RegisterTemplate(v8::Isolate* isolate) {
+WrapperInfo* MyObject2::GetWrapperInfo() {
+  return &kWrapperInfo;
+}
+
+
+class MyObjectBlink : public Wrappable {
+ public:
+  MyObjectBlink() {
+  }
+  static WrapperInfo kWrapperInfo;
+  virtual WrapperInfo* GetWrapperInfo() OVERRIDE;
+};
+
+WrapperInfo MyObjectBlink::kWrapperInfo = { kEmbedderBlink };
+
+WrapperInfo* MyObjectBlink::GetWrapperInfo() {
+  return &kWrapperInfo;
+}
+
+
+void RegisterTemplates(v8::Isolate* isolate) {
   PerIsolateData* data = PerIsolateData::From(isolate);
   DCHECK(data->GetObjectTemplate(&MyObject::kWrapperInfo).IsEmpty());
 
   v8::Handle<v8::ObjectTemplate> templ = ObjectTemplateBuilder(isolate)
       .SetProperty("value", &MyObject::value, &MyObject::set_value)
       .Build();
   templ->SetInternalFieldCount(kNumberOfInternalFields);
   data->SetObjectTemplate(&MyObject::kWrapperInfo, templ);
+
+  templ = v8::ObjectTemplate::New(isolate);
+  templ->SetInternalFieldCount(kNumberOfInternalFields);
+  data->SetObjectTemplate(&MyObject2::kWrapperInfo, templ);
+
+  templ = v8::ObjectTemplate::New(isolate);
+  templ->SetInternalFieldCount(kNumberOfInternalFields);
+  data->SetObjectTemplate(&MyObjectBlink::kWrapperInfo, templ);
 }
 
 typedef V8Test WrappableTest;
 
 TEST_F(WrappableTest, WrapAndUnwrap) {
   v8::Isolate* isolate = instance_->isolate();
   v8::HandleScope handle_scope(isolate);
 
-  RegisterTemplate(isolate);
+  RegisterTemplates(isolate);
   Handle<MyObject> obj = MyObject::Create(isolate);
 
   v8::Handle<v8::Value> wrapper = ConvertToV8(isolate, obj.get());
   EXPECT_FALSE(wrapper.IsEmpty());
 
-  MyObject* unwrapped = 0;
+  MyObject* unwrapped = NULL;
   EXPECT_TRUE(ConvertFromV8(isolate, wrapper, &unwrapped));
   EXPECT_EQ(obj.get(), unwrapped);
 }
 
+TEST_F(WrappableTest, UnwrapFailures) {
+  v8::Isolate* isolate = instance_->isolate();
+  v8::HandleScope handle_scope(isolate);
+
+  RegisterTemplates(isolate);
+
+  // Something that isn't an object.
+  v8::Handle<v8::Value> thing = v8::Number::New(42);
+  MyObject* unwrapped = NULL;
+  EXPECT_FALSE(ConvertFromV8(isolate, thing, &unwrapped));
+  EXPECT_FALSE(unwrapped);
+
+  // An object that's not wrapping anything.
+  thing = v8::Object::New(isolate);
+  EXPECT_FALSE(ConvertFromV8(isolate, thing, &unwrapped));
+  EXPECT_FALSE(unwrapped);
+
+  // An object that's wrapping a C++ object from Blink.
+  thing.Clear();
+  thing = ConvertToV8(isolate, new MyObjectBlink());
+  EXPECT_FALSE(ConvertFromV8(isolate, thing, &unwrapped));
+  EXPECT_FALSE(unwrapped);
+
+  // An object that's wrapping a C++ object of the wrong type.
+  thing.Clear();
+  thing = ConvertToV8(isolate, new MyObject2());
+  EXPECT_FALSE(ConvertFromV8(isolate, thing, &unwrapped));
+  EXPECT_FALSE(unwrapped);
+}
+
 TEST_F(WrappableTest, GetAndSetProperty) {
   v8::Isolate* isolate = instance_->isolate();
   v8::HandleScope handle_scope(isolate);
 
-  RegisterTemplate(isolate);
+  RegisterTemplates(isolate);
   gin::Handle<MyObject> obj = MyObject::Create(isolate);
 
   obj->set_value(42);
   EXPECT_EQ(42, obj->value());
 
   v8::Handle<v8::String> source = StringToV8(isolate,
       "(function (obj) {"
       "   if (obj.value !== 42) throw 'FAIL';"
       "   else obj.value = 191; })");
   EXPECT_FALSE(source.IsEmpty());
@@ skipping 10 matching lines @@
   };
   func->Call(v8::Undefined(isolate), 1, argv);
   EXPECT_FALSE(try_catch.HasCaught());
   EXPECT_EQ("", try_catch.GetStackTrace());
 
   EXPECT_EQ(191, obj->value());
 }
 
 }  // namespace
 }  // namespace gin