Deprecate FunctionCallbackInfo::Holder

Use FunctionCallbackInfo::This instead.

See http://crbug.com/333672197.

Author: kkoopa

nan_callbacks_12_inl.h

@@ -111,7 +111,19 @@ class FunctionCallbackInfo {
   }
 #endif
   inline v8::Local<v8::Value> Data() const { return data_; }
-  inline v8::Local<v8::Object> Holder() const { return info_.Holder(); }
+  inline v8::Local<v8::Object> Holder() const {
+#if defined(V8_MAJOR_VERSION) &&                                               \
+    (V8_MAJOR_VERSION > 12 ||                                                  \
+     (V8_MAJOR_VERSION == 12 &&                                                \
+      (defined(V8_MINOR_VERSION) &&                                            \
+       (V8_MINOR_VERSION > 5 ||                                                \
+        (V8_MINOR_VERSION == 5 && defined(V8_BUILD_NUMBER) &&                  \
+         V8_BUILD_NUMBER >= 214)))))
+    return info_.This();
+#else
+    return info_.Holder();
+#endif
+  }
   inline bool IsConstructCall() const { return info_.IsConstructCall(); }
   inline int Length() const { return info_.Length(); }
   inline v8::Local<v8::Value> operator[](int i) const { return info_[i]; }

nan_callbacks_pre_12_inl.h

@@ -140,7 +140,9 @@ class FunctionCallbackInfo {
     return args_.Callee();
   }
   inline v8::Local<v8::Value> Data() const { return data_; }
-  inline v8::Local<v8::Object> Holder() const { return args_.Holder(); }
+  inline v8::Local<v8::Object> Holder() const {
+    return args_.Holder();
+  }
   inline bool IsConstructCall() const { return args_.IsConstructCall(); }
   inline int Length() const { return args_.Length(); }
   inline v8::Local<v8::Value> operator[](int i) const { return args_[i]; }

test/js/accessors-test.js

@@ -11,7 +11,7 @@ const test     = require('tap').test
     , bindings = require('bindings')({ module_root: testRoot, bindings: 'accessors' });
 
 test('accessors', function (t) {
-  t.plan(7)
+  t.plan(9)
   var settergetter = bindings.create()
   t.equal(settergetter.prop1, 'this is property 1')
   t.ok(settergetter.prop2 === '')
@@ -28,5 +28,8 @@ test('accessors', function (t) {
   t.equal(derived.prop1, 'this is property 1')
   derived.prop2 = 'setting a new value'
   t.equal(derived.prop2, 'setting a new value')
-  t.equal(settergetter.prop2, 'setting a new value')
+  t.equal(settergetter.prop2, 'setting a value')
+  settergetter.prop2 = 'setting another value'
+  t.equal(settergetter.prop2, 'setting another value')
+  t.equal(derived.prop2, 'setting a new value')
 })

test/js/methodswithdata-test.js

@@ -11,28 +11,30 @@ const test     = require('tap').test
     , bindings = require('bindings')({ module_root: testRoot, bindings: 'methodswithdata' })
 
 test('SetMethod with data', function (t) {
-    t.plan(1);
-    t.ok(bindings.testWithData());
+  t.plan(1);
+  t.ok(bindings.testWithData());
 });
 
 test('accessors with data', function (t) {
-    t.plan(7)
-    var settergetter = bindings.create()
-    t.equal(settergetter.prop1, 'this is property 1')
-    t.ok(settergetter.prop2 === '')
-    settergetter.prop2 = 'setting a value'
-    t.equal(settergetter.prop2, 'setting a value')
-    t.equal(settergetter.log(),
-      'New()\n' +
-      'Prop1:GETTER(this is property 1)\n' +
-      'Prop2:GETTER()\n' +
-      'Prop2:SETTER(setting a value)\n' +
-      'Prop2:GETTER(setting a value)\n'
-    )
-    var derived = Object.create(settergetter)
-    t.equal(derived.prop1, 'this is property 1')
-    derived.prop2 = 'setting a new value'
-    t.equal(derived.prop2, 'setting a new value')
-    t.equal(settergetter.prop2, 'setting a new value')
-  })
-  
+  t.plan(9)
+  var settergetter = bindings.create()
+  t.equal(settergetter.prop1, 'this is property 1')
+  t.ok(settergetter.prop2 === '')
+  settergetter.prop2 = 'setting a value'
+  t.equal(settergetter.prop2, 'setting a value')
+  t.equal(settergetter.log(),
+    'New()\n' +
+    'Prop1:GETTER(this is property 1)\n' +
+    'Prop2:GETTER()\n' +
+    'Prop2:SETTER(setting a value)\n' +
+    'Prop2:GETTER(setting a value)\n'
+  )
+  var derived = Object.create(settergetter)
+  t.equal(derived.prop1, 'this is property 1')
+  derived.prop2 = 'setting a new value'
+  t.equal(derived.prop2, 'setting a new value')
+  t.equal(settergetter.prop2, 'setting a value')
+  settergetter.prop2 = 'setting another value'
+  t.equal(settergetter.prop2, 'setting another value')
+  t.equal(derived.prop2, 'setting a new value')
+})