Add documentation for platform/bindings

third_party/WebKit/Source/platform/bindings/ScopedPersistent.h

Index: third_party/WebKit/Source/platform/bindings/ScopedPersistent.h
diff --git a/third_party/WebKit/Source/platform/bindings/ScopedPersistent.h b/third_party/WebKit/Source/platform/bindings/ScopedPersistent.h
index 2cf5ae2a90504f486215c7519d55a4903841680e..b61ab924734067d9733ead85d0dc37b886c47bae 100644
--- a/third_party/WebKit/Source/platform/bindings/ScopedPersistent.h
+++ b/third_party/WebKit/Source/platform/bindings/ScopedPersistent.h
@@ -40,6 +40,7 @@
 
 namespace blink {
 
+// Holds a persistent handle to a V8 object.

[haraken, 2017/04/29 02:39:11]

Add that:

- developers should use ScopedPersistent instead of using v8::Persistent
directly
- developers must be super careful when introducing a (non-weak)
ScopedPersistent since it has a high risk of producing memory leaks. Ask
blink-reviews-bindings@ for review.

[jbroman, 2017/04/29 04:11:30]

Aside: What does ScopedPersistent do that v8::Global doesn't? Should we
deprecate ScopedPersistent at some point?

[adithyas, 2017/05/01 17:41:16]

Done.

 template <typename T>
 class ScopedPersistent {
   USING_FAST_MALLOC(ScopedPersistent);