diff options
author | Adrian Roos <roosa@google.com> | 2015-03-09 19:10:15 +0100 |
---|---|---|
committer | Adrian Roos <roosa@google.com> | 2015-03-09 19:14:33 +0100 |
commit | a43fd03b9b23f9ad0deae9ee69ee80aaa1433cdb (patch) | |
tree | 70a46df079bec1612274fdc1732e24dc342cfc87 /core | |
parent | f855b0d2d0c0c157d657668fcb98097e2aca097e (diff) | |
download | frameworks_base-a43fd03b9b23f9ad0deae9ee69ee80aaa1433cdb.zip frameworks_base-a43fd03b9b23f9ad0deae9ee69ee80aaa1433cdb.tar.gz frameworks_base-a43fd03b9b23f9ad0deae9ee69ee80aaa1433cdb.tar.bz2 |
Notify TrustAgentService when device policy configuration no longer applies
Also updates the outdated javadoc to match the current API.
Bug: 19500731
Change-Id: I7976bbcc815b84e168718672377a963ffea68c51
Diffstat (limited to 'core')
-rw-r--r-- | core/java/android/service/trust/TrustAgentService.java | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/core/java/android/service/trust/TrustAgentService.java b/core/java/android/service/trust/TrustAgentService.java index 706b934..a3178e2 100644 --- a/core/java/android/service/trust/TrustAgentService.java +++ b/core/java/android/service/trust/TrustAgentService.java @@ -122,12 +122,14 @@ public class TrustAgentService extends Service { case MSG_CONFIGURE: ConfigurationData data = (ConfigurationData) msg.obj; boolean result = onConfigure(data.options); - try { - synchronized (mLock) { - mCallback.onConfigureCompleted(result, data.token); + if (data.token != null) { + try { + synchronized (mLock) { + mCallback.onConfigureCompleted(result, data.token); + } + } catch (RemoteException e) { + onError("calling onSetTrustAgentFeaturesEnabledCompleted()"); } - } catch (RemoteException e) { - onError("calling onSetTrustAgentFeaturesEnabledCompleted()"); } break; case MSG_TRUST_TIMEOUT: @@ -203,7 +205,7 @@ public class TrustAgentService extends Service { * PersistableBundle)}. * <p>Agents that support configuration options should overload this method and return 'true'. * - * @param options bundle containing all options or null if none. + * @param options The aggregated list of options or an empty list if no restrictions apply. * @return true if the {@link TrustAgentService} supports configuration options. */ public boolean onConfigure(List<PersistableBundle> options) { |