commit 7037efa4f297cdc270f2f088f7a5e3e12e8cd29d
author Mark Tabry <mtab@google.com> Mon Jun 10 22:37:28 2019 +0000
committer Mark Tabry <mtab@google.com> Mon Jun 10 23:02:45 2019 +0000
tree f3979e6e463c28db780865d4c5b199953c1592ab
parent 44f225a48a3ed2d31f40179f25f15bcf63c78b24

Revert "Register VmsPublisheService with VmsClientManager on construction time."

This reverts commit 44f225a48a3ed2d31f40179f25f15bcf63c78b24.

Reason for revert: VmsPublisherServiceTest cleanup was missing from original CL.
Bug: 134708512

Change-Id: I4864ce28a7cf932139deeb8da1c14678330e082a

service/src/com/android/car/VmsPublisherService.java

diff --git a/service/src/com/android/car/VmsPublisherService.java b/service/src/com/android/car/VmsPublisherService.java
index 647b102..3384e3a 100644
--- a/service/src/com/android/car/VmsPublisherService.java
+++ b/service/src/com/android/car/VmsPublisherService.java
@@ -119,14 +119,16 @@
         mContext = context;
         mClientManager = clientManager;
         mBrokerService = brokerService;
+    }
+
+    @Override
+    public void init() {
         mClientManager.registerConnectionListener(this);
     }
 
     @Override
-    public void init() {}
-
-    @Override
     public void release() {
+        mClientManager.unregisterConnectionListener(this);
         mPublisherProxies.values().forEach(PublisherProxy::unregister);
         mPublisherProxies.clear();
     }