Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit b545c20f authored by Kweku Adams's avatar Kweku Adams Committed by Automerger Merge Worker
Browse files

Merge "Log EJ status to statsd." into sc-dev am: e3585e10 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/13877231

Change-Id: I74adf1da6fabf6e27883be9423f8d33a89dd52c2
parents 980c4020 e3585e10

apex/jobscheduler/service/java/com/android/server/job/JobSchedulerService.java

+4 −2
Original line number Diff line number Diff line
@@ -1046,7 +1046,7 @@ public class JobSchedulerService extends com.android.server.SystemService 
            FrameworkStatsLog.write_non_chained(FrameworkStatsLog.SCHEDULED_JOB_STATE_CHANGED,

                    uId, null, jobStatus.getBatteryName(),

                    FrameworkStatsLog.SCHEDULED_JOB_STATE_CHANGED__STATE__SCHEDULED,

                    JobProtoEnums.STOP_REASON_CANCELLED, jobStatus.getStandbyBucket(),

                    JobProtoEnums.STOP_REASON_UNKNOWN, jobStatus.getStandbyBucket(),

                    jobStatus.getJobId(),

                    jobStatus.hasChargingConstraint(),

                    jobStatus.hasBatteryNotLowConstraint(),
@@ -1055,7 +1055,9 @@ public class JobSchedulerService extends com.android.server.SystemService 
                    jobStatus.hasDeadlineConstraint(),

                    jobStatus.hasIdleConstraint(),

                    jobStatus.hasConnectivityConstraint(),

                    jobStatus.hasContentTriggerConstraint());

                    jobStatus.hasContentTriggerConstraint(),

                    jobStatus.isRequestedExpeditedJob(),

                    /* isRunningAsExpeditedJob */ false);



            // If the job is immediately ready to run, then we can just immediately

            // put it in the pending list and try to schedule it.  This is especially

apex/jobscheduler/service/java/com/android/server/job/JobServiceContext.java

+6 −2
Original line number Diff line number Diff line
@@ -322,7 +322,9 @@ public final class JobServiceContext implements ServiceConnection { 
                    job.hasDeadlineConstraint(),

                    job.hasIdleConstraint(),

                    job.hasConnectivityConstraint(),

                    job.hasContentTriggerConstraint());

                    job.hasContentTriggerConstraint(),

                    job.isRequestedExpeditedJob(),

                    job.shouldTreatAsExpeditedJob());

            try {

                mBatteryStats.noteJobStart(job.getBatteryName(), job.getSourceUid());

            } catch (RemoteException e) {
@@ -904,7 +906,9 @@ public final class JobServiceContext implements ServiceConnection { 
                completedJob.hasDeadlineConstraint(),

                completedJob.hasIdleConstraint(),

                completedJob.hasConnectivityConstraint(),

                completedJob.hasContentTriggerConstraint());

                completedJob.hasContentTriggerConstraint(),

                completedJob.isRequestedExpeditedJob(),

                completedJob.startedAsExpeditedJob);

        try {

            mBatteryStats.noteJobFinish(mRunningJob.getBatteryName(), mRunningJob.getSourceUid(),

                    legacyStopReason);

apex/jobscheduler/service/java/com/android/server/job/controllers/JobStatus.java

+3 −2
Original line number Diff line number Diff line
@@ -142,13 +142,14 @@ public final class JobStatus { 
     * (Atom #21)

     * * CONSTRAINT_BACKGROUND_NOT_RESTRICTED can be inferred with BatterySaverModeStateChanged

     * (Atom #20)

     * * CONSTRAINT_STORAGE_NOT_LOW can be inferred with LowStorageStateChanged (Atom #130)

     */

    private static final int STATSD_CONSTRAINTS_TO_LOG = CONSTRAINT_CONTENT_TRIGGER

            | CONSTRAINT_DEADLINE

            | CONSTRAINT_IDLE

            | CONSTRAINT_STORAGE_NOT_LOW

            | CONSTRAINT_TIMING_DELAY

            | CONSTRAINT_WITHIN_QUOTA;

            | CONSTRAINT_WITHIN_QUOTA

            | CONSTRAINT_WITHIN_EXPEDITED_QUOTA;



    // TODO(b/129954980)

    private static final boolean STATS_LOG_ENABLED = false;