diff options
| author | tim@chromium.org <tim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-25 05:51:40 +0000 |
|---|---|---|
| committer | tim@chromium.org <tim@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-25 05:51:40 +0000 |
| commit | 633a10a8162d96014ecb1ddcd01e1bb1e584d6c1 (patch) | |
| tree | 7ac914d112e57486fbb2f600616aa5efc50512b2 /mojo/mojo.gyp | |
| parent | 251e8c5b5d3bc50cc3a07d2088a5ea739d7aabc6 (diff) | |
| download | chromium_src-633a10a8162d96014ecb1ddcd01e1bb1e584d6c1.zip chromium_src-633a10a8162d96014ecb1ddcd01e1bb1e584d6c1.tar.gz chromium_src-633a10a8162d96014ecb1ddcd01e1bb1e584d6c1.tar.bz2 | |
mojo: don't leak ServiceConnectors if they are re-Added to ServiceRegistry
It seems easy enough to AddService<T> for multiple T where T implements a given interface Foo; I did it by accident in a test. This patch keeps the overwrite behavior the same, but deletes the old ServiceConnector.
Also add a test to verify ServiceConnectors are deleted from ServiceRegistry in all cases.
BUG=none
Review URL: https://codereview.chromium.org/340653004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279617 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'mojo/mojo.gyp')
| -rw-r--r-- | mojo/mojo.gyp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/mojo/mojo.gyp b/mojo/mojo.gyp index 6e3acbf..53e7324 100644 --- a/mojo/mojo.gyp +++ b/mojo/mojo.gyp @@ -45,6 +45,7 @@ 'mojo_network_service', 'mojo_pepper_container_app', 'mojo_profile_service', + 'mojo_public_application_unittests', 'mojo_public_test_utils', 'mojo_public_bindings_unittests', 'mojo_public_environment_unittests', |