Mojo C++ bindings: reduce references to AssociatedGroupController.

Mojo C++ bindings: reduce references to AssociatedGroupController.

If an interface doesn't pass associated kinds, there is no need to store a ref to AssociatedGroupController in the serialization context.

That shouldn't have any effect on the interface pointer side, because all those references will be cleared when pipe disconnection happens or the interface pointer is destroyed. However on the binding side, if we don't do this, as long as someone holds on the any method callback given to the impl, the AssociatedGroupController will be kept alive.

BUG=651103
Committed: https://crrev.com/3fa6ea22828a127d5ee91afe01553531f0932880
Cr-Commit-Position: refs/heads/master@{#431353}

[yzshen1, 2016-11-10 19:52:54]

The CQ bit was checked by yzshen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 19:53:27]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yzshen1, 2016-11-10 19:56:42]

The CQ bit was checked by yzshen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 19:57:10]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yzshen1, 2016-11-10 19:58:20]

yzshen@chromium.org changed reviewers:
+ rockot@chromium.org

[yzshen1, 2016-11-10 19:58:20]

Hi, Ken.

This is a speculative improvement for the memory regression on Android One. I am
still looking for other possible improvements.

Please take a look. Thanks!

[Ken Rockot(use gerrit already), 2016-11-10 20:05:52]

lgtm

[yzshen1, 2016-11-10 20:06:30]

The CQ bit was unchecked by yzshen@chromium.org

[yzshen1, 2016-11-10 20:06:30]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2016-11-10 20:06:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-10 21:14:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-10 21:23:38]

Description was changed from

==========
Mojo C++ bindings: reduce references to AssociatedGroupController.

If an interface doesn't pass associated kinds, there is no need to store a ref
to AssociatedGroupController in the serialization context.

That shouldn't have any effect on the interface pointer side, because all those
references will be cleared when pipe disconnection happens or the interface
pointer is destroyed. However on the binding side, if we don't do this, as long
as someone holds on the any method callback given to the impl, the
AssociatedGroupController will be kept alive.

BUG=651103
==========

to

==========
Mojo C++ bindings: reduce references to AssociatedGroupController.

If an interface doesn't pass associated kinds, there is no need to store a ref
to AssociatedGroupController in the serialization context.

That shouldn't have any effect on the interface pointer side, because all those
references will be cleared when pipe disconnection happens or the interface
pointer is destroyed. However on the binding side, if we don't do this, as long
as someone holds on the any method callback given to the impl, the
AssociatedGroupController will be kept alive.

BUG=651103

Committed: https://crrev.com/3fa6ea22828a127d5ee91afe01553531f0932880
Cr-Commit-Position: refs/heads/master@{#431353}
==========

[commit-bot: I haz the power, 2016-11-10 21:23:39]

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