PPAPI: Release renderer Var refcount when returning to plugin

PPAPI: Release renderer Var refcount when returning to plugin

BUG=451624
R=raymes@chromium.org

Committed: https://crrev.com/64d576c36c2cf39d3ca14dba6bccfdf76291439e
Cr-Commit-Position: refs/heads/master@{#313355}

[raymes, 2015-01-27 00:12:44]

lgtm! Thanks for doing this!

https://codereview.chromium.org/874703003/diff/1/ppapi/proxy/host_var_seriali...
File ppapi/proxy/host_var_serialization_rules.cc (right):

https://codereview.chromium.org/874703003/diff/1/ppapi/proxy/host_var_seriali...
ppapi/proxy/host_var_serialization_rules.cc:49: // See
PluginVarSerialization::ReceivePassRef for an example. We don't need
nit: PluginVarSerializationRules?

https://codereview.chromium.org/874703003/diff/1/ppapi/proxy/serialized_var_u...
File ppapi/proxy/serialized_var_unittest.cc (right):

https://codereview.chromium.org/874703003/diff/1/ppapi/proxy/serialized_var_u...
ppapi/proxy/serialized_var_unittest.cc:284: EXPECT_EQ(-1,
var_tracker().GetRefCountForObject(string_var));
Should we test the object var case too? Or does that get tested somewhere else?

[dmichael (off chromium), 2015-01-27 19:32:42]

The CQ bit was checked by dmichael@chromium.org

[commit-bot: I haz the power, 2015-01-27 19:33:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/874703003/20001

[commit-bot: I haz the power, 2015-01-27 21:00:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-27 21:02:19]

Patchset 2 (id:??) landed as
https://crrev.com/64d576c36c2cf39d3ca14dba6bccfdf76291439e
Cr-Commit-Position: refs/heads/master@{#313355}