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

Unified Diff: ppapi/c/pp_var.h

Issue 18599005: [PPAPI] Added PP_VARTYPE_RESOURCE as a PP_VarType enum value. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Fix tests; slightly better behaviour if user passes a resource to a message. Created 7 years, 3 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
Index: ppapi/c/pp_var.h
diff --git a/ppapi/c/pp_var.h b/ppapi/c/pp_var.h
index 2138f93c2bddea9e7c6116dd846581cb8a8e78ed..68a24cd8071f95bf228388c19d5c5f5b5502c2de 100644
--- a/ppapi/c/pp_var.h
+++ b/ppapi/c/pp_var.h
@@ -3,7 +3,7 @@
* found in the LICENSE file.
*/
-/* From pp_var.idl modified Mon Feb 11 15:41:10 2013. */
+/* From pp_var.idl modified Tue Sep 3 13:35:39 2013. */
#ifndef PPAPI_C_PP_VAR_H_
#define PPAPI_C_PP_VAR_H_
@@ -76,7 +76,13 @@ typedef enum {
* contiguously. See PPB_VarArrayBuffer_Dev for functions special to
* ArrayBuffer vars.
*/
- PP_VARTYPE_ARRAY_BUFFER = 9
+ PP_VARTYPE_ARRAY_BUFFER = 9,
+ /**
+ * Resources are not currently supported but will be added in future
+ * revisions. These objects are reference counted, so AddRef and Release must
+ * be used properly to avoid memory leaks.
+ */
+ PP_VARTYPE_RESOURCE = 10
} PP_VarType;
PP_COMPILE_ASSERT_SIZE_IN_BYTES(PP_VarType, 4);
/**
@@ -112,8 +118,8 @@ union PP_VarValue {
double as_double;
/**
* If <code>type</code> is <code>PP_VARTYPE_STRING</code>,
- * <code>PP_VARTYPE_OBJECT</code>, <code>PP_VARTYPE_ARRAY</code>, or
- * <code>PP_VARTYPE_DICTIONARY</code>,
+ * <code>PP_VARTYPE_OBJECT</code>, <code>PP_VARTYPE_ARRAY</code>,
+ * <code>PP_VARTYPE_DICTIONARY</code>, or <code>PP_VARTYPE_RESOURCE</code>,
* <code>as_id</code> represents the value of this <code>PP_Var</code> as
* an opaque handle assigned by the browser. This handle is guaranteed
* never to be 0, so a module can initialize this ID to 0 to indicate a

Powered by Google App Engine
This is Rietveld 408576698