PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove unnecessary check canScheduleExactAlarms() as Settings always

Browse Source 
has the permission.

Test: make RunSettingsRoboTests
Bug: 256123455
Change-Id: I5089494164581465da9533671fc414c7bee1196f
This commit is contained in:
Kuan Wang
2022-11-07 13:49:48 +08:00
parent fe26b89394
commit d81a6ceefd
4 changed files with 0 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/com/android/settings/fuelgauge/batteryusage/PeriodicJobManager.java
View File

@@ -73,10 +73,6 @@ public final class PeriodicJobManager {
// Cancels the previous alert job and schedules the next one.
final PendingIntent pendingIntent = getPendingIntent();
cancelJob(pendingIntent);
if (!canScheduleExactAlarms()) {
Log.w(TAG, "cannot schedule exact alarm job");
return;
}
// Uses UTC time to avoid scheduler is impacted by different timezone.
final long triggerAtMillis = getTriggerAtMillis(Clock.systemUTC());
mAlarmManager.setExactAndAllowWhileIdle(
@@ -111,13 +107,4 @@ public final class PeriodicJobManager {
broadcastIntent,
PendingIntent.FLAG_CANCEL_CURRENT | PendingIntent.FLAG_IMMUTABLE);
}
private boolean canScheduleExactAlarms() {
return canScheduleExactAlarms(mAlarmManager);
}
/** Whether we can schedule exact alarm or not? */
public static boolean canScheduleExactAlarms(AlarmManager alarmManager) {
return alarmManager.canScheduleExactAlarms();
}
}

1
tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BootBroadcastReceiverTest.java
View File

@@ -58,7 +58,6 @@ public final class BootBroadcastReceiverTest {
mContext = ApplicationProvider.getApplicationContext();
mPeriodicJobManager = PeriodicJobManager.getInstance(mContext);
mShadowAlarmManager = shadowOf(mContext.getSystemService(AlarmManager.class));
ShadowAlarmManager.setCanScheduleExactAlarms(true);
mReceiver = new BootBroadcastReceiver();
// Inserts fake data into database for testing.

10
tests/robotests/src/com/android/settings/fuelgauge/batteryusage/PeriodicJobManagerTest.java
View File

@@ -48,7 +48,6 @@ public final class PeriodicJobManagerTest {
mContext = ApplicationProvider.getApplicationContext();
mPeriodicJobManager = PeriodicJobManager.getInstance(mContext);
mShadowAlarmManager = shadowOf(mContext.getSystemService(AlarmManager.class));
ShadowAlarmManager.setCanScheduleExactAlarms(true);
}
@After
@@ -68,15 +67,6 @@ public final class PeriodicJobManagerTest {
assertThat(alarm.operation).isNotNull();
}
@Test
public void refreshJob_withoutPermission_notRefreshesAlarmJob() {
ShadowAlarmManager.setCanScheduleExactAlarms(false);
mPeriodicJobManager.refreshJob();
assertThat(mShadowAlarmManager.peekNextScheduledAlarm()).isNull();
}
@Test
public void getTriggerAtMillis_withoutOffset_returnsExpectedResult() {
long timeSlotUnit = PeriodicJobManager.DATA_FETCH_INTERVAL_MINUTE;

1
tests/robotests/src/com/android/settings/fuelgauge/batteryusage/PeriodicJobReceiverTest.java
View File

@@ -58,7 +58,6 @@ public final class PeriodicJobReceiverTest {
mContext = ApplicationProvider.getApplicationContext();
mPeriodicJobManager = PeriodicJobManager.getInstance(mContext);
mShadowAlarmManager = shadowOf(mContext.getSystemService(AlarmManager.class));
ShadowAlarmManager.setCanScheduleExactAlarms(true);
mReceiver = new PeriodicJobReceiver();
// Inserts fake data into database for testing.