From d4c45e81046d4f638d8fca4c0c341b912139aeee Mon Sep 17 00:00:00 2001 From: vadimt Date: Thu, 24 Jan 2019 13:52:49 -0800 Subject: [PATCH] Removing querying whether DWB time limit is for a group This API is removed for Q. Bug: 118319143 Test: Manual Change-Id: I1605ba14a914bfe018c4a2f6d974afaea7e51e05 --- quickstep/res/values/strings.xml | 4 ---- .../quickstep/views/DigitalWellBeingToast.java | 18 ++++++------------ 2 files changed, 6 insertions(+), 16 deletions(-) diff --git a/quickstep/res/values/strings.xml b/quickstep/res/values/strings.xml index 7c4795694b..0c741a140f 100644 --- a/quickstep/res/values/strings.xml +++ b/quickstep/res/values/strings.xml @@ -62,8 +62,4 @@ %1$s left today - - - %1$s left for group \ No newline at end of file diff --git a/quickstep/src/com/android/quickstep/views/DigitalWellBeingToast.java b/quickstep/src/com/android/quickstep/views/DigitalWellBeingToast.java index aafd725cb2..9ad750b1b3 100644 --- a/quickstep/src/com/android/quickstep/views/DigitalWellBeingToast.java +++ b/quickstep/src/com/android/quickstep/views/DigitalWellBeingToast.java @@ -77,7 +77,6 @@ public final class DigitalWellBeingToast extends LinearLayout { Utilities.THREAD_POOL_EXECUTOR.execute(() -> { long appUsageLimitTimeMs = -1; long appRemainingTimeMs = -1; - boolean isGroupLimit = true; try { final Method getAppUsageLimit = LauncherApps.class.getMethod( @@ -95,8 +94,6 @@ public final class DigitalWellBeingToast extends LinearLayout { invoke(usageLimit); appRemainingTimeMs = (long) appUsageLimitClass.getMethod("getUsageRemaining"). invoke(usageLimit); - isGroupLimit = (boolean) appUsageLimitClass.getMethod("isGroupLimit"). - invoke(usageLimit); } } catch (Exception e) { // Do nothing @@ -104,14 +101,13 @@ public final class DigitalWellBeingToast extends LinearLayout { final long appUsageLimitTimeMsFinal = appUsageLimitTimeMs; final long appRemainingTimeMsFinal = appRemainingTimeMs; - final boolean isGroupLimitFinal = isGroupLimit; post(() -> { if (appUsageLimitTimeMsFinal < 0) { setVisibility(GONE); } else { setVisibility(VISIBLE); - mText.setText(getText(appRemainingTimeMsFinal, isGroupLimitFinal)); + mText.setText(getText(appRemainingTimeMsFinal)); mImage.setImageResource(appRemainingTimeMsFinal > 0 ? R.drawable.hourglass_top : R.drawable.hourglass_bottom); } @@ -119,9 +115,7 @@ public final class DigitalWellBeingToast extends LinearLayout { callback.call( appUsageLimitTimeMsFinal >= 0 && appRemainingTimeMsFinal <= 0 ? 0 : 1, getContentDescriptionForTask( - task, appUsageLimitTimeMsFinal, - appRemainingTimeMsFinal, - isGroupLimitFinal)); + task, appUsageLimitTimeMsFinal, appRemainingTimeMsFinal)); }); }); } @@ -185,12 +179,12 @@ public final class DigitalWellBeingToast extends LinearLayout { duration, FormatWidth.NARROW, R.string.shorter_duration_less_than_one_minute); } - private String getText(long remainingTime, boolean isGroupLimit) { + private String getText(long remainingTime) { final Resources resources = getResources(); return (remainingTime <= 0) ? resources.getString(R.string.app_in_grayscale) : resources.getString( - isGroupLimit ? R.string.time_left_for_group : R.string.time_left_for_app, + R.string.time_left_for_app, getShorterReadableDuration(Duration.ofMillis(remainingTime))); } @@ -214,12 +208,12 @@ public final class DigitalWellBeingToast extends LinearLayout { } private String getContentDescriptionForTask( - Task task, long appUsageLimitTimeMs, long appRemainingTimeMs, boolean isGroupLimit) { + Task task, long appUsageLimitTimeMs, long appRemainingTimeMs) { return appUsageLimitTimeMs >= 0 ? getResources().getString( R.string.task_contents_description_with_remaining_time, task.titleDescription, - getText(appRemainingTimeMs, isGroupLimit)) : + getText(appRemainingTimeMs)) : task.titleDescription; } }