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

Commit 8d6ab2f9 authored by Kweku Adams's avatar Kweku Adams
Browse files

Don't penalize app for slow app binding. 

The system may be busy when trying to bind to the app, so don't penalize
it for slow binding. Now, we'll retain the job that was slow to bind and
stop triggering an ANR.

Bug: 292090044
Bug: 303533836
Test: atest CtsJobSchedulerTestCases:JobSchedulingTest
Change-Id: Ida52d63c5e9710d9d590388d20fceb4fa8f2e0d5
(cherry picked from commit 71fdfd48)
Merged-In: Ida52d63c5e9710d9d590388d20fceb4fa8f2e0d5
parent ef2cbcd7

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

+3 −3
Original line number Diff line number Diff line
@@ -1327,13 +1327,13 @@ public final class JobServiceContext implements ServiceConnection { 
    private void handleOpTimeoutLocked() {

        switch (mVerb) {

            case VERB_BINDING:

                onSlowAppResponseLocked(/* reschedule */ false, /* updateStopReasons */ true,

                // The system may have been too busy. Don't drop the job or trigger an ANR.

                onSlowAppResponseLocked(/* reschedule */ true, /* updateStopReasons */ true,

                        /* texCounterMetricId */

                        "job_scheduler.value_cntr_w_uid_slow_app_response_binding",

                        /* debugReason */ "timed out while binding",

                        /* anrMessage */ "Timed out while trying to bind",

                        CompatChanges.isChangeEnabled(ANR_PRE_UDC_APIS_ON_SLOW_RESPONSES,

                            mRunningJob.getUid()));

                        /* triggerAnr */ false);

                break;

            case VERB_STARTING:

                // Client unresponsive - wedged or failed to respond in time. We don't really