From 61cbb25b12e21233ec6fc7691d850e4c7bd434d7 Mon Sep 17 00:00:00 2001 From: Brad Parker Date: Mon, 20 Mar 2017 14:46:43 +0000 Subject: [PATCH] add callback for permission request result, style nits --- .../retroactivity/RetroActivityCommon.java | 28 +++++++++++++++++-- 1 file changed, 25 insertions(+), 3 deletions(-) diff --git a/pkg/android/phoenix/src/com/retroarch/browser/retroactivity/RetroActivityCommon.java b/pkg/android/phoenix/src/com/retroarch/browser/retroactivity/RetroActivityCommon.java index 48a858976f..df4c1122a2 100644 --- a/pkg/android/phoenix/src/com/retroarch/browser/retroactivity/RetroActivityCommon.java +++ b/pkg/android/phoenix/src/com/retroarch/browser/retroactivity/RetroActivityCommon.java @@ -35,12 +35,15 @@ public class RetroActivityCommon extends RetroActivityLocation private boolean addPermission(List permissionsList, String permission) { - if (checkSelfPermission(permission) != PackageManager.PERMISSION_GRANTED) { + if (checkSelfPermission(permission) != PackageManager.PERMISSION_GRANTED) + { permissionsList.add(permission); + // Check for Rationale Option if (!shouldShowRequestPermissionRationale(permission)) return false; - } + } + return true; } @@ -79,7 +82,8 @@ public class RetroActivityCommon extends RetroActivityLocation message = message + ", " + permissionsNeeded.get(i); showMessageOKCancel(message, - new DialogInterface.OnClickListener() { + new DialogInterface.OnClickListener() + { @Override public void onClick(DialogInterface dialog, int which) { @@ -101,6 +105,24 @@ public class RetroActivityCommon extends RetroActivityLocation } } + @Override + public void onRequestPermissionsResult(int requestCode, String[] permissions, int[] grantResults) + { + super.onRequestPermissionsResult(requestCode, permissions, grantResults); + + for (int i = 0; i < permissions.length; i++) + { + if(grantResults[i] == PackageManager.PERMISSION_GRANTED) + { + Log.i("RetroActivity", "Permission: " + permissions[i] + " was granted."); + } + else + { + Log.i("RetroActivity", "Permission: " + permissions[i] + " was not granted."); + } + } + } + public boolean isAndroidTV() { Configuration config = getResources().getConfiguration();