More work on ServiceProviderImpl.

More work on ServiceProviderImpl.

* Remove the old AddService<I>().
* Rename AddServiceNew<I>() to AddService<I>().
* Add a getter for the ConnectionContext.
* Add a AddServiceForName() that takes a ServiceConnector.
* Add RemoveService<I>() and RemoveServiceForName().

The additions are basically required to replace the current
ServiceProvider implementation in stuff in ServiceRegistry with just a
ServiceProviderImpl.

R=vardhan@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/90ee9b339c3304a5bb84bbf75e4b926aa1577af8

[vardhan, 2016-05-13 00:26:16]

lgtm

[viettrungluu, 2016-05-13 15:59:11]

Description was changed from

==========
More work on ServiceProviderImpl.

* Remove the old AddService<I>().
* Rename AddServiceNew<I>() to AddService<I>().
* Add a getter for the ConnectionContext.
* Add a AddServiceForName() that takes a ServiceConnector.
* Add RemoveService<I>() and RemoveServiceForName().

The additions are basically required to replace the current
ServiceProvider implementation in stuff in ServiceRegistry with just a
ServiceProviderImpl.

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

to

==========
More work on ServiceProviderImpl.

* Remove the old AddService<I>().
* Rename AddServiceNew<I>() to AddService<I>().
* Add a getter for the ConnectionContext.
* Add a AddServiceForName() that takes a ServiceConnector.
* Add RemoveService<I>() and RemoveServiceForName().

The additions are basically required to replace the current
ServiceProvider implementation in stuff in ServiceRegistry with just a
ServiceProviderImpl.

R=vardhan@google.com

Committed:
https://chromium.googlesource.com/external/mojo/+/90ee9b339c3304a5bb84bbf75e4...
==========

[viettrungluu, 2016-05-13 15:59:13]

Committed patchset #3 (id:40001) manually as
90ee9b339c3304a5bb84bbf75e4b926aa1577af8 (presubmit successful).