Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(164)

Unified Diff: ppapi/api/ppp_messaging.idl

Issue 16140011: Don't send PP_Vars/V8 values with cycles across PostMessage (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ppapi/api/ppb_messaging.idl ('k') | ppapi/c/ppb_messaging.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/api/ppp_messaging.idl
diff --git a/ppapi/api/ppp_messaging.idl b/ppapi/api/ppp_messaging.idl
index 150f7712c15656867cb43df7e4ce3f4f5da8de6c..83c2010382bed71084e34224480f79dd879449cd 100644
--- a/ppapi/api/ppp_messaging.idl
+++ b/ppapi/api/ppp_messaging.idl
@@ -36,10 +36,9 @@ interface PPP_Messaging {
*
* When converting JavaScript arrays, any object properties whose name
* is not an array index are ignored. When passing arrays and objects, the
- * entire reference graph will be converted and transferred, including
- * reference cycles if they exist. Since <code>PP_Var</code>s are ref-counted,
- * the author of the plugin must take care if they expect to receive vars with
- * cycles. Cycles must be manually broken to correctly release the vars.
+ * entire reference graph will be converted and transferred. If the reference
+ * graph has cycles, the message will not be sent and an error will be logged
+ * to the console.
*
* The following JavaScript code invokes <code>HandleMessage</code>, passing
* the module instance on which it was invoked, with <code>message</code>
« no previous file with comments | « ppapi/api/ppb_messaging.idl ('k') | ppapi/c/ppb_messaging.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698