Don't use soon-to-be-deprecated V8 Api (#182)

Namely v8::FunctionCallbackInfo::Holder(), one should use This()
method instead.
See https://crrev.com/c/5444829 for details.

Author: isheludko

src/README.md

@@ -906,9 +906,9 @@ class from C++.
 The JavaScript object can be accessed as a `v8::Local<v8::Object>` by using
 `self->object()`, given a `BaseObject` named `self`.
 
-Accessing a `BaseObject` from a `v8::Local<v8::Object>` (frequently that is
-`args.This()` in a [binding function][]) can be done using
-the `Unwrap<T>(obj)` function, where `T` is a subclass of `BaseObject`.
+Accessing a `BaseObject` from a `v8::Local<v8::Object>` (that is `args.This()`
+in a [binding function][]) can be done using the `Unwrap<T>(obj)` function,
+where `T` is a subclass of `BaseObject`.
 A helper for this is the `ASSIGN_OR_RETURN_UNWRAP` macro that returns from the
 current function if unwrapping fails (typically that means that the `BaseObject`
 has been deleted earlier).