Mojo: introducing a thread safe interface pointer.

mojo/public/tools/bindings/generators/cpp_templates/interface_proxy_declaration.tmpl

Index: mojo/public/tools/bindings/generators/cpp_templates/interface_proxy_declaration.tmpl
diff --git a/mojo/public/tools/bindings/generators/cpp_templates/interface_proxy_declaration.tmpl b/mojo/public/tools/bindings/generators/cpp_templates/interface_proxy_declaration.tmpl
index cfc4ef6770d7d3db24188718b1dac39cbed8aa0e..7c20a2392529e74ee151620201de57eca845ecaf 100644
--- a/mojo/public/tools/bindings/generators/cpp_templates/interface_proxy_declaration.tmpl
+++ b/mojo/public/tools/bindings/generators/cpp_templates/interface_proxy_declaration.tmpl
@@ -11,11 +11,10 @@ class {{export_attribute}} {{interface.name}}Proxy
   void {{method.name}}({{interface_macros.declare_request_params("", method)}}) override;
 {%- endfor %}
 
-  mojo::internal::SerializationContext* serialization_context() {
-    return &serialization_context_;
+  void set_group_controller(mojo::AssociatedGroupController* group_controller) {

[yzshen1, 2016/11/15 18:53:46]

I think it is preferred to avoid passing raw pointer of ref-counted object as
input?

We could use a scoped_refptr as input and then do group_controller_ =
std::move(group_controller) to avoid extra addref/release.

WDYT?

[Jay Civelli, 2016/11/15 19:36:38]

Sounds reasonable, done.

+    group_controller_ = group_controller;
   }
-
  private:
   mojo::MessageReceiverWithResponder* receiver_;
-  mojo::internal::SerializationContext serialization_context_;
+  scoped_refptr<mojo::AssociatedGroupController> group_controller_;
 };