Make ServiceRegistry (which implements) ApplicationConnection just wrap a ServiceProviderImpl.

Make ServiceRegistry (which implements) ApplicationConnection just wrap a ServiceProviderImpl.

At this point, I could make ApplicationConnection not an interface, and
just rename ServiceRegistry to ApplicationConnection (basically). I
won't bother, because I plan on deleting
ApplicationConnection/ServiceRegistry ASAP.

I also added a ServiceProviderImpl getter to ApplicationConnection. My
plan of action is:
* Make everyone currently using ApplicationConnection just use stuff via
  the getter. (In particular, this means converting all the current uses
  of ApplicationConnection::AddService<I>()/InterfaceFactory<I>.)
* At that point, replacing ApplicationConnection with just
  ServiceProviderImpl will be easy.

R=vardhan@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/721c34a0c8b9cde24940a2cbbb6173897b79ed6b

[vardhan, 2016-05-13 18:06:59]

lgtm

[viettrungluu, 2016-05-13 18:12:54]

Description was changed from

==========
Make ServiceRegistry (which implements) ApplicationConnection just wrap a
ServiceProviderImpl.

At this point, I could make ApplicationConnection not an interface, and
just rename ServiceRegistry to ApplicationConnection (basically). I
won't bother, because I plan on deleting
ApplicationConnection/ServiceRegistry ASAP.

I also added a ServiceProviderImpl getter to ApplicationConnection. My
plan of action is:
* Make everyone currently using ApplicationConnection just use stuff via
  the getter. (In particular, this means converting all the current uses
  of ApplicationConnection::AddService<I>()/InterfaceFactory<I>.)
* At that point, replacing ApplicationConnection with just
  ServiceProviderImpl will be easy.

R=vardhan@google.com
==========

to

==========
Make ServiceRegistry (which implements) ApplicationConnection just wrap a
ServiceProviderImpl.

At this point, I could make ApplicationConnection not an interface, and
just rename ServiceRegistry to ApplicationConnection (basically). I
won't bother, because I plan on deleting
ApplicationConnection/ServiceRegistry ASAP.

I also added a ServiceProviderImpl getter to ApplicationConnection. My
plan of action is:
* Make everyone currently using ApplicationConnection just use stuff via
  the getter. (In particular, this means converting all the current uses
  of ApplicationConnection::AddService<I>()/InterfaceFactory<I>.)
* At that point, replacing ApplicationConnection with just
  ServiceProviderImpl will be easy.

R=vardhan@google.com

Committed:
https://chromium.googlesource.com/external/mojo/+/721c34a0c8b9cde24940a2cbbb6...
==========

[viettrungluu, 2016-05-13 18:12:56]

Committed patchset #1 (id:1) manually as
721c34a0c8b9cde24940a2cbbb6173897b79ed6b (presubmit successful).