From 58d8b2c889389e098253f574144098fef46f21e6 Mon Sep 17 00:00:00 2001 From: Peter Fong Date: Mon, 1 Jul 2019 11:37:07 -0700 Subject: Minor review comment fixes --- .../android/mobly/snippet/bundled/WifiManagerSnippet.java | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'src/main/java/com/google/android/mobly') diff --git a/src/main/java/com/google/android/mobly/snippet/bundled/WifiManagerSnippet.java b/src/main/java/com/google/android/mobly/snippet/bundled/WifiManagerSnippet.java index 3ad85a2..8d8f19a 100644 --- a/src/main/java/com/google/android/mobly/snippet/bundled/WifiManagerSnippet.java +++ b/src/main/java/com/google/android/mobly/snippet/bundled/WifiManagerSnippet.java @@ -69,7 +69,7 @@ public class WifiManagerSnippet implements Snippet { mWifiManager = (WifiManager) mContext.getApplicationContext().getSystemService(Context.WIFI_SERVICE); - elevatePermissionIfRequired(); + adaptShellPermissionIfRequired(); } @Rpc( @@ -395,12 +395,13 @@ public class WifiManagerSnippet implements Snippet { /** * Elevates permission as require for proper wifi controls. * + * Starting in Android Q (29), additional restrictions are added for wifi operation. See + * below Android Q privacy changes for additional details. + * https://developer.android.com/preview/privacy/camera-connectivity + * * @throws Throwable */ - private void elevatePermissionIfRequired() throws Throwable { - // Starting in Android Q (29), additional restrictions are added for wifi operation. See - // below Android Q privacy changes for additional details. - // https://developer.android.com/preview/privacy/camera-connectivity + private void adaptShellPermissionIfRequired() throws Throwable { if (mContext.getApplicationContext().getApplicationInfo().targetSdkVersion >= 29 && Build.VERSION.SDK_INT >= 29) { Log.d("Elevating permission require to enable support for wifi operation in Android Q+"); -- cgit v1.2.3