Add a ScopedPPVar class for helping to track a reference to a base PP_Var.

Add a ScopedPPVar class for helping to track a reference to a base PP_Var.

This will be used in unit tests and (possibly in the future) son proxy implementation code. This is basically copied from ScopedPPResource.

TEST=none
BUG=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=142982

[brettw, 2012-06-18 04:16:47]

Not a high priority, should be easy. I'm cleaving off parts of my bigger change
to land separately.

[dmichael (off chromium), 2012-06-18 20:09:16]

http://codereview.chromium.org/10562033/diff/1/ppapi/shared_impl/scoped_pp_var.h
File ppapi/shared_impl/scoped_pp_var.h (right):

http://codereview.chromium.org/10562033/diff/1/ppapi/shared_impl/scoped_pp_va...
ppapi/shared_impl/scoped_pp_var.h:20: ScopedPPVar(const PP_Var& v);
probably should be explicit?

http://codereview.chromium.org/10562033/diff/1/ppapi/shared_impl/scoped_pp_va...
ppapi/shared_impl/scoped_pp_var.h:36: operator PP_Var() const { return var_; }
This might be dangerous. Should we force people to think about whether or not
they're passing a reference by making them use get() explicitly when they aren't
passing a reference?

[brettw, 2012-06-19 04:36:02]

New snap up, PTAL

[dmichael (off chromium), 2012-06-19 14:34:55]

lgtm

[commit-bot: I haz the power, 2012-06-19 16:17:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/brettw@chromium.org/10562033/8001

[commit-bot: I haz the power, 2012-06-19 17:36:09]

Change committed as 142982