http://cgit.collabora.com/git/user/cassidy/telepathy-glib/log/?h=next-tls-cert-76175

 - You're missing the _TP_AVAILABLE_IN_ for tp_client_factory_ensure_tls_certificate().

 - "Returns: (transfer full): a new TLS certificate proxy" --> "a reference to a" is better IMO, like others ensure_foo. I see you did the same for protocol.

 - tp_client_factory_ensure_tls_certificate() is missing the variant g_return_val_if_fail(). Like tp_proxy_get_factory(conn_or_chan) == self to be sure we don't mix proxies from different factories.

 - I think we should not add the vfuncs, but they can be removed later with the rest. So that's not merge blocker for now.

(In reply to comment #2)
>  - You're missing the _TP_AVAILABLE_IN_ for
> tp_client_factory_ensure_tls_certificate().
> 
>  - "Returns: (transfer full): a new TLS certificate proxy" --> "a reference
> to a" is better IMO, like others ensure_foo. I see you did the same for
> protocol.
> 
>  - tp_client_factory_ensure_tls_certificate() is missing the variant
> g_return_val_if_fail(). Like tp_proxy_get_factory(conn_or_chan) == self to
> be sure we don't mix proxies from different factories.

All fixed in both branches.

+1

Merged to next.