diff options
author | Stephen Bird <sbird@cyngn.com> | 2016-04-28 16:19:30 -0700 |
---|---|---|
committer | Stephen Bird <sbird@cyngn.com> | 2016-04-28 16:24:18 -0700 |
commit | b528bb96b0668938b42894aea0e9c5427c6eb75f (patch) | |
tree | 2a5ee0a44e93037e5bca9577ca8408d4cda31eb6 | |
parent | f401a4dc15cdb88fa47e49266cc309345a60f428 (diff) | |
download | packages_apps_PhoneCommon-b528bb96b0668938b42894aea0e9c5427c6eb75f.tar.gz packages_apps_PhoneCommon-b528bb96b0668938b42894aea0e9c5427c6eb75f.tar.bz2 packages_apps_PhoneCommon-b528bb96b0668938b42894aea0e9c5427c6eb75f.zip |
DataSubscription: Broadcast if we have no plugins
If we have no plugins then we need to broadcast to
our subscribers so they do not worry about them.
Ticket: CD-602
Change-Id: I2db52417d335bece3a66f08384331fa9cc6f4648
(cherry picked from commit f21fbc17de836937d836ebcf068e44dab3e08ab5)
-rw-r--r-- | src-ambient/com/android/phone/common/ambient/AmbientDataSubscription.java | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/src-ambient/com/android/phone/common/ambient/AmbientDataSubscription.java b/src-ambient/com/android/phone/common/ambient/AmbientDataSubscription.java index 3b30ccb..4234e77 100644 --- a/src-ambient/com/android/phone/common/ambient/AmbientDataSubscription.java +++ b/src-ambient/com/android/phone/common/ambient/AmbientDataSubscription.java @@ -69,11 +69,16 @@ public abstract class AmbientDataSubscription<M> { @Override public void onResult(Result result) { List<ComponentName> installedPlugins = getPluginComponents(result); - for (ComponentName cn : installedPlugins) { - ArrayList<TypedPendingResult> apiCallbacks = new ArrayList<>(); - getPluginInfo().put(cn, getNewModObject(cn)); - requestedModInfo(apiCallbacks, cn); - executeAll(apiCallbacks, cn); + if (installedPlugins.size() != 0) { + for (ComponentName cn : installedPlugins) { + ArrayList<TypedPendingResult> apiCallbacks = new ArrayList<>(); + getPluginInfo().put(cn, getNewModObject(cn)); + requestedModInfo(apiCallbacks, cn); + executeAll(apiCallbacks, cn); + } + } else { + // We want to tell our subscribers that we have no plugins to worry about + broadcast(); } } |