If a job doesn't say that it needs network, then there's no reason to
ensure that the app has access to network while the job is running.
Currently, the network restriction bypass flags are only given to
expedited or user-initiated jobs. Don't include the flags when the job
doesn't require connectivity to avoid accidental network usage/leakage.

Bug: 297106511
Bug: 299329948
Bug: 299346198
Test: atest CtsJobSchedulerTestCases:ConnectivityConstraintTest
Test: atest CtsJobSchedulerTestCases:ExpeditedJobTest
Change-Id: I9c632595f27655a977d6b084760839ce02c8c07b

Developers are informed that expedited jobs can be stopped earlier than
regular jobs and so shouldn't use them for long pieces of work.
The standard timeout is 3 minutes for EJs. There's little reason to let
them run longer than the normal 10 minutes, and letting them run up to
30 minutes may give developers the wrong impression and increases the
potential for accidental power issues. Lower the maximum execution limit
of EJs to 10 minutes (the base execution limit of regular jobs) to be
more in line with what EJs should be used for and prevent additional
issues.

Bug: 284512488
Bug: 297106511
Bug: 299329948
Bug: 299346198
Test: atest FrameworksMockingServicesTests:JobSchedulerServiceTest
Change-Id: Ia4950f0801f2881a34976987f735e5ab7f493ee1

Create a separate JobScheduler idle value by charging + battery-not-low
state so that we can have different values when the device is charging
compared to when it's on battery.

Bug: 236261941
Bug: 297106511
Bug: 299329948
Test: atest CtsJobSchedulerTestCases:IdleConstraintTest
Test: atest frameworks/base/services/tests/mockingservicestests/src/com/android/server/job
Test: atest frameworks/base/services/tests/servicestests/src/com/android/server/job
Change-Id: I16f41f05f7ffe5ab7fd1f660e0e08ea7544ff921

startOp, finishOp methods

This allows us to reduce the number of parameters being passed around
and also enables easy addition of new parameters such as device id.

Note that this change is unflagged and is meant to be a pure refactor.
This is preparatory CL and no new methods are being added to AppOpsManager yet, but will be needed
to allow clients to pass in device id.

Bug: 299160174
Test: atest CtsAppOpsTestCases CtsAppOps2TestCases
Change-Id: I591b1dc7d7e4af0371c2d1d80b06d858e7213a94

This reverts commit eb72459a.

Reason for revert: The plan is to apply it to jobs by default.

Bug: 236261941
Bug: 299346198
Test: atest FrameworksMockingServicesTests:ConnectivityControllerTest
Test: atest FrameworksMockingServicesTests:FlexibilityControllerTest
Test: atest FrameworksMockingServicesTests:JobStatusTest
Change-Id: Ie9a5643caf6e6b68642580b92b10c05a70bd2998

The job bias is meant to be an indication of the job type and the state
the app was in when the job was scheduled. This can only be determined
by the system and certain privileged apps. The job bias can affect some
things such as which system health factors are taken into consideration
when deciding when to run a job. Given all this, the bias should only be
set by the system or certain privileged apps. Note the required permission
on the hidden API and enforce that the calling app has the permission
when the job is scheduled.

Bug: 300477393
Test: atest CtsJobSchedulerTestCases:JobInfoTest
Test: atest CtsSyncManagerTestCases
Test: atest DownloadManagerApi28Test
Test: atest FrameworksServicesTests:BiasSchedulingTest
Change-Id: I1cb37eafc42f961dee9d2e7bc7d496a31dff31f4

during Job execution

Test: manual testing on device done by building
Bug: 255393346
Change-Id: If876aed542e8edf282ac36ff53979919bf41ce90

Don't relax the prefetch connectivity constraint requirement unless the
battery is also not low.

Bug: 299329948
Test: atest FrameworksMockingServicesTests:PrefetchControllerTest
Test: atest FrameworksMockingServicesTests:ConnectivityControllerTest
Change-Id: I4b2c0905fb15277c63ae57284a2139858998d68b

1. Remove the job ID from the wakelock tag.
2. Lazily load the tag when needed to reduce memory usage of jobs
   waiting for constraints to be satisfied.
3. Remove redundant log line.

Bug: 273758274
Bug: 302348192
Bug: 307556230
Test: atest CtsJobSchedulerTestCases
Test: Manually remove output of `adb shell dumpsys jobscheduler`
Change-Id: If8547d23ced90b17225ab8efd758ac735655687e

Use UserHandle.parseUserArg() when parsing adb command flags to enable
parsing "cur" and other special user strings.

Bug: 282063891
Test: atest CtsJobSchedulerTestCases
Test: `adb shell cmd jobscheduler run --user cur com.android.settings 2131427352`
Change-Id: Ibe06f536f1de760a5604d067b6c1894f756a17c6

The uid parameter to scheduleAsPackage is the calling UID. It was
incorrectly being used as the source UID in the statsd logging, causing
the end metrics to have the wrong information. This fixes the UID
logging and renames the input parameter to avoid further confusion.

Bug: 138239687
Test: N/A
Change-Id: Ifdfe2ffbc7b5f8b8fe74291a1ec54068541f740b

When loading from disk, skip files such as backup & temp files created
during the AtomicFile writing process to avoid incorrectly reading
duplicate jobs from them.

Bug: 289062813
Bug: 305169670
Test: atest FrameworksServicesTests:JobStoreTest
Change-Id: I28a180e2b24357219d34be6a22c785311be6e72e

Various issues here, but the most recurring one is referencing SystemApi
from public javadoc. Remove or rewrite such references.

Also common is trying to @link IntDefs. Usually this is not necessary,
as the presence of IntDef in the @param/@return auto-generates relevant
documentation.

Bug: 303184203
Test: m docs
Change-Id: Ie59e61881c45c73fbbe08250493bd35eca268ef6

Now that exact alarms are denied to newer app installs for apps targeting T, the deny list is obsolete and the overhead of maintaining it is not worthwhile.

Test: atest CtsAlarmManagerTestCases
Test: atest FrameworksMockingServicesTests:com.android.server.alarm

Bug: 304846238
Change-Id: I4376e5ddbb2e470f3e3d690dd0866f4b470f455b

Bug: 299329948
Change-Id: If1f6ef1a950ee93c2cb4413205c87a9454734573
Test: N/A

If the persisted files have somehow accumulated duplicate versions of
the same job (identified by uid-namespace-jobId), then drop the
duplicates and only keep one.

Bug: 289062813
Bug: 296890885
Test: atest JobStoreTest
Change-Id: I3179261309031801f0506cf905206376e59af1fe
(cherry picked from commit c8fbd937)
Merged-In: I3179261309031801f0506cf905206376e59af1fe

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

Reduce the prefetch job launch time threshold to one hour so we will
only consider running a prefetch job at most one hour before the app is
estimated to be launched.

Bug: 295046507
Test: atest FrameworksMockingServicesTests:PrefetchControllerTest
Change-Id: Ia3d4d049fb44b8f12418c987425949b60b37bd65

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

Make it possible to increase the light idle time linearly instead of
exponentially.

Bug: 299329948
Test: atest DeviceIdleTest
Test: atest FrameworksMockingServicesTests:DeviceIdleControllerTest
Change-Id: I4e7c07fa1d2653446ae5617f11fbd26ec39438ab

Only let prefetch jobs use the opportunistic quota when the device
is on charger to reduce excess battery drain. The opportunistic quota
usage behavior is optional and not required for prefetch jobs, so we can
disable it without impacting app or developer expectations.

Bug: 295046507
Bug: 299329948
Test: atest FrameworksMockingServicesTests:ConnectivityControllerTest
Test: atest FrameworksMockingServicesTests:PrefetchControllerTest
Change-Id: Idc137279fea1100f53ecf20279153fe92fdb5617

Platform_testing large test annotation is for
host side tests and restricted, device tests should use
androidx annotation.

Bug: 293837621
Test: presubmit
Change-Id: I835c4c837fa2074c937d7068f17c75467fb213dc

The statsd logging was incorrectly logging for only the source app.
Including the calling app will enable better metrics and debugging.

Bug: 138239687
Test: statsd_testdrive 8 while running CtsSyncManagerTestsCases
Test: statsd_testdrive 150 while running CtsSyncManagerTestsCases
Change-Id: I9ba3097135b1f11368df800b043d730d780f3a33

Bug: 294435720

Test: build

Change-Id: I3a788a7202827623b50d25b0a71bd6237a04d47c

Bug: 294435720

Test: atest FrameworksMockingServicesTests: com.android.server.DeviceIdleControllerTest

Change-Id: I908b0a8db05ef0feb19c5ecab47c1065295d17a4

The background thread is prone to severe delays in processing time near
boot, which causes pull timeouts and event queue overflow in statsd. The
vast majority of pullers use the DIRECT_EXECUTOR to execute pulls in the
incoming binder thread, so move these pulled atoms to do the same thing.
More context is in the bug and linked docs.

Bug: 298133385
Test: verified via boot traces that the pull durations are shorter and
that the pulls no longer happen in the background thread

Change-Id: I67a3d583cc13d9547dfe7108897e95f4a5238d26
Merged-In: I67a3d583cc13d9547dfe7108897e95f4a5238d26
(cherry picked from commit 5c85f29b)

Bug: 298968496
Test: PackageInstallerArchiveTest.java
Change-Id: Ic9669cc76e7a37455d773d00268130426b2a5165

Tear down the Agent after each test so that it doesn't continue to
operate on the Handler thread.

Bug: 299645803
Test: atest --rerun-until-failure 50 com.android.server.tare.AgentTest
Change-Id: I07f4742a1a438ce61919bf5d75a75484fb3815db

Platform_testing LargeTest annotation is for
host side tests, device
tests should use androidx annotation.

Bug: 293837621
Test: build
Change-Id: I8ea6f256b542a4ed0030b3d366c37f6fdafa6b3b

Bug: 290817480

Test: Build

Change-Id: Icc16f2af5226efd4e74d82fa31f08e2cd200348c

Currently only sends an intent to the responsible app store.

Test: PackageArchiverTest, PackagerArchiverServiceTest
Bug: 290777297
Change-Id: I8fc1eba023a108d7bca5819565933b4a77dd7c37

The background thread is prone to severe delays in processing time near
boot, which causes pull timeouts and event queue overflow in statsd. The
vast majority of pullers use the DIRECT_EXECUTOR to execute pulls in the
incoming binder thread, so move these pulled atoms to do the same thing.
More context is in the bug and linked docs.

Bug: 298133385
Test: verified via boot traces that the pull durations are shorter and
that the pulls no longer happen in the background thread

Change-Id: I67a3d583cc13d9547dfe7108897e95f4a5238d26

See https://source.android.com/setup/contribute/respectful-code for reference

For this round, the fixes are only applied to the following to minimize breaking dependencies:
  * comments (excluding javaDoc annotations)
	* private constants
	* private functions
	* parameters within functions

BYPASS_INCLUSIVE_LANGUAGE_REASON=Just updating a few select inclusive language violations.
No-Typo-Check: Changes focused on inclusive language violations.
BUG: 295342157
Change-Id: I690b29e1c26dc97cfbdee274ba61c76229b568e0

Any LocalServices object that is reused in a class should be cached locally
so that we avoid repeated lookups within the LocalServices map,
especially during tight loops.

Bug: 295877092
Test: atest CtsJobSchedulerTestCases
Test: atest FrameworksMockingServicesTests:JobStatusTest
Test: atest FrameworksMockingServicesTests:QuotaControllerTest
Change-Id: Ie8d90a0f8aebbb23138613095859e1f9ce1b207f

If the persisted files have somehow accumulated duplicate versions of
the same job (identified by uid-namespace-jobId), then drop the
duplicates and only keep one.

Bug: 289062813
Bug: 296890885
Test: atest JobStoreTest
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:c8fbd937940b83944492c4ccaf2830611285999a)
Merged-In: I3179261309031801f0506cf905206376e59af1fe
Change-Id: I3179261309031801f0506cf905206376e59af1fe

JobServiceContext will leak when fail to bindService.

issue: 291625922

test: monkey
Change-Id: Icca4aa1f1fe32b6f39d18f7ee819cc682cac6f18

If the persisted files have somehow accumulated duplicate versions of
the same job (identified by uid-namespace-jobId), then drop the
duplicates and only keep one.

Bug: 289062813
Bug: 296890885
Test: atest JobStoreTest
Change-Id: I3179261309031801f0506cf905206376e59af1fe

If the persisted files have somehow accumulated duplicate versions of
the same job (identified by uid-namespace-jobId), then drop the
duplicates and only keep one.

Bug: 289062813
Bug: 296890885
Test: atest JobStoreTest
Change-Id: I3179261309031801f0506cf905206376e59af1fe

See https://source.android.com/setup/contribute/respectful-code for reference

For this round, the fixes are only applied to the following to minimize breaking dependencies:
  * comments (excluding javaDoc annotations)
	* private constants
	* private functions
	* parameters within functions

BYPASS_INCLUSIVE_LANGUAGE_REASON=Just updating a few select inclusive language violations.
No-Typo-Check: Changes focused on inclusive language violations.
BUG: 295342157

Change-Id: Ib6c6616202d2795174ff253f863173857f9c79c9

Note how JobScheduler behaves when apps don't respond in a timely manner
to onStartJob() and onStopJob().

Bug: 19536175
Test: `m offline-sdk-docs` and view page at out/target/common/docs/offline-sdk/reference/android/app/job/JobService.html
Change-Id: Ib5586801addd3ff83ce19476f98e9ef1cee4e51d