Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Remove wtf." into tm-dev

Browse Source
This commit is contained in:
Kweku Adams
2022-04-27 20:58:53 +00:00
committed by Android (Google) Code Review
parent e5b570be58 ca02f07824
commit 2998b5aaa7
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
apex/jobscheduler/service/java/com/android/server/job/JobSchedulerService.java
View File

@@ -1749,8 +1749,13 @@ public class JobSchedulerService extends com.android.server.SystemService
if (!removed) { if (!removed) {
// We never create JobStatus objects for the express purpose of removing them, and this // We never create JobStatus objects for the express purpose of removing them, and this
// method is only ever called for jobs that were saved in the JobStore at some point, // method is only ever called for jobs that were saved in the JobStore at some point,
// so if we can't find it, something went seriously wrong. // so if we can't find it, something may be wrong. As of Android T, there is a
Slog.wtfStack(TAG, "Job didn't exist in JobStore"); // legitimate code path where removed is false --- when an actively running job is
// cancelled (eg. via JobScheduler.cancel() or the app scheduling a new job with the
// same job ID), we remove it from the JobStore and tell the JobServiceContext to stop
// running the job. Once the job stops running, we then call this method again.
// TODO: rework code so we don't intentionally call this method twice for the same job
Slog.w(TAG, "Job didn't exist in JobStore");
} }
if (mReadyToRock) { if (mReadyToRock) {
for (int i = 0; i < mControllers.size(); i++) { for (int i = 0; i < mControllers.size(); i++) {