[api] Add documentation for PropertyCallbackInfo.

include/v8.h

Index: include/v8.h
diff --git a/include/v8.h b/include/v8.h
index c93ee238585f5a86527532eb6e927ac7906a9ccb..88d71f1f9021c6a7ef35f49b90ea0353787188ef 100644
--- a/include/v8.h
+++ b/include/v8.h
@@ -3236,12 +3236,46 @@ class FunctionCallbackInfo {
 template<typename T>
 class PropertyCallbackInfo {
  public:
+  /**
+   * \return The isolate of the property access.
+   */
   V8_INLINE Isolate* GetIsolate() const;
+
+  /**
+   * \return The data set in the configuration, i.e., in
+   * `NamedPropertyHandlerConfiguration` or
+   * `IndexedPropertyHandlerConfiguration.`
+   */
   V8_INLINE Local<Value> Data() const;
+
+  /**
+   * \return The receiver, i.e., the object on which the property access was
+   * intercepted.

[jochen (gone - plz use gerrit), 2016/08/23 12:43:26]

Not sure I understand.

so assuming you have x and x's prototype is y, and y has an interceptor. then
This() would be x, but Holder() would be y.

[Franzi, 2016/08/23 13:11:01]

Done.

+   * In `x.a`, the receiver is `x`.
+   */
   V8_INLINE Local<Object> This() const;
+
+  /**
+   * \return The object that is the target property, i.e., in `x.a`, the
+   * holder is `a`.

[jochen (gone - plz use gerrit), 2016/08/23 12:43:26]

see above

[Franzi, 2016/08/23 13:11:01]

Done.

+   */
   V8_INLINE Local<Object> Holder() const;
+
+  /**
+   * \return The return value of the callback.
+   * Can be used to define the return value by calling Set().
+   * \code
+   * info.GetReturnValue().Set(...)
+   * \endcode
+   */
   V8_INLINE ReturnValue<T> GetReturnValue() const;
+
+  /**
+   * \return True, if the intercepted function should
+   * throw if an error occurs. Usually, `true` corresponds to `'use strict.'`
+   */
   V8_INLINE bool ShouldThrowOnError() const;
+
   // This shouldn't be public, but the arm compiler needs it.
   static const int kArgsLength = 7;