commit 885894a0e13d62026a1d611ddd43b53a9e40c734
author Didi She <didis@google.com> Tue Apr 27 10:40:57 2021 -0700
committer Didi She <didis@google.com> Tue Apr 27 10:52:24 2021 -0700
tree 31a7a161783f665a27157b5effbc5c5696bff6fd
parent ec71c0fec5726eef0aa8a768ea370c8c9e22f739

Do not change protocol callback when loading subscriptionHandler

In subscription persistence feature, we didn't persist binding, and
shared same binding between client and handler for mutual subscription.
The handler is only registered on the binding after the binding has been
prepared. The only meaningful event in the callback BindingFailed.
We should use the same protocol callback with subscriptionClient.

src/lib/profiles/data-management/Current/SubscriptionHandler.cpp

diff --git a/src/lib/profiles/data-management/Current/SubscriptionHandler.cpp b/src/lib/profiles/data-management/Current/SubscriptionHandler.cpp
index a32ab9d..ad888a6 100644
--- a/src/lib/profiles/data-management/Current/SubscriptionHandler.cpp
+++ b/src/lib/profiles/data-management/Current/SubscriptionHandler.cpp
@@ -1785,7 +1785,6 @@
 
     // Capture the binding and arrange to receive event callbacks.
     mBinding->AddRef();
-    mBinding->SetProtocolLayerCallback(BindingEventCallback, this);
 
     mBytesOffloaded = 0;