[PPAPI] Updated documentation for PP_VarType enum constants.

[PPAPI] Updated documentation for PP_VarType enum constants.

Wrote proper documentation for PP_VARTYPE_ARRAY and PP_VARTYPE_DICTIONARY. These
previously said that they are not currently supported, but they are.

For PP_VARTYPE_STRING, added note that AddRef and Release must be used.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221654

[yzshen1, 2013-09-03 18:04:54]

https://codereview.chromium.org/23757014/diff/1/ppapi/api/pp_var.idl
File ppapi/api/pp_var.idl (right):

https://codereview.chromium.org/23757014/diff/1/ppapi/api/pp_var.idl#newcode47
ppapi/api/pp_var.idl:47: * <code>PPB_Var</code> interface. These objects are
reference counted, so
If you would like to mention about ref counting, all types below, not just
string/array/dictionary should have the same comment.

[Matt Giuca, 2013-09-04 00:12:53]

https://codereview.chromium.org/23757014/diff/1/ppapi/api/pp_var.idl
File ppapi/api/pp_var.idl (right):

https://codereview.chromium.org/23757014/diff/1/ppapi/api/pp_var.idl#newcode47
ppapi/api/pp_var.idl:47: * <code>PPB_Var</code> interface. These objects are
reference counted, so
Done. The reason I did that was because the existing ARRAY and DICTIONARY
documentation mentioned it. Do you actually want all of these to have the same
text?

[yzshen1, 2013-09-04 00:18:15]

https://codereview.chromium.org/23757014/diff/1/ppapi/api/pp_var.idl
File ppapi/api/pp_var.idl (right):

https://codereview.chromium.org/23757014/diff/1/ppapi/api/pp_var.idl#newcode47
ppapi/api/pp_var.idl:47: * <code>PPB_Var</code> interface. These objects are
reference counted, so
On 2013/09/04 00:12:53, Matt Giuca wrote:
> Done. The reason I did that was because the existing ARRAY and DICTIONARY
> documentation mentioned it. Do you actually want all of these to have the same
> text?
Yes, mentioning it for string/array/dictionary but not object/array_buffer
sounds like the latter two don't require ref counting.

You could talk about ref counting above "enum PP_VarType". That way you don't
need to repeat the text 5 times.

BTW, I cannot see your latest patch.

[Matt Giuca, 2013-09-04 00:29:31]

Sorry, Reitveld was having a moment...

It's fixed now. Do you want me to put the refcounting docs at the top instead?

[yzshen1, 2013-09-04 07:11:00]

On 2013/09/04 00:29:31, Matt Giuca wrote:
> Sorry, Reitveld was having a moment...
> 
> It's fixed now. Do you want me to put the refcounting docs at the top instead?
I am fine with either way.

LGTM
Thanks!

[raymes, 2013-09-06 05:06:14]

On 2013/09/04 07:11:00, yzshen1 wrote:
> On 2013/09/04 00:29:31, Matt Giuca wrote:
> > Sorry, Reitveld was having a moment...
> > 
> > It's fixed now. Do you want me to put the refcounting docs at the top
instead?
> I am fine with either way.
> 
> LGTM
> Thanks!

lgtm!

[commit-bot: I haz the power, 2013-09-06 05:33:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mgiuca@chromium.org/23757014/29001

[commit-bot: I haz the power, 2013-09-06 08:57:30]

Change committed as 221654