commit 651617b09c228d317421967946261c04c90767a6
author Hai Zhang <zhanghai@google.com> Wed Mar 06 20:40:39 2019 -0800
committer Hai Zhang <zhanghai@google.com> Fri Mar 08 14:33:41 2019 -0800
tree 6d67dfa77db54ec75c24c08969af56107019b997
parent d49498c48019be0a88ff53a5b2e0980feb8aa17e

Use a boolean Consumer instead of RoleManagerCallback.

And use RemoteCallback for it under the hood. This removes one AIDL
interface and allows using lambda, while aligning with what
PermissionControllerService does.

Bug: 127691087
Test: manual
Change-Id: Ied4c6b6fd0ba51ac30313db36552768ca5cf0592

src/com/android/phone/EmergencyAssistanceHelper.java

diff --git a/src/com/android/phone/EmergencyAssistanceHelper.java b/src/com/android/phone/EmergencyAssistanceHelper.java
index a98d81d..7e6c282 100644
--- a/src/com/android/phone/EmergencyAssistanceHelper.java
+++ b/src/com/android/phone/EmergencyAssistanceHelper.java
@@ -17,7 +17,6 @@
 package com.android.phone;
 
 import android.app.role.RoleManager;
-import android.app.role.RoleManagerCallback;
 import android.content.ComponentName;
 import android.content.Context;
 import android.content.Intent;
@@ -143,12 +142,8 @@
         try {
             context.getSystemService(RoleManager.class).addRoleHolderAsUser(
                     RoleManager.ROLE_EMERGENCY, pkgName, 0, Process.myUserHandle(),
-                    AsyncTask.THREAD_POOL_EXECUTOR, new RoleManagerCallback() {
-                        @Override
-                        public void onSuccess() {
-                        }
-                        @Override
-                        public void onFailure() {
+                    AsyncTask.THREAD_POOL_EXECUTOR, successful -> {
+                        if (!successful) {
                             Log.e(TAG, "Failed to set emergency default app.");
                         }
                     });