Pass an empty list to PerformBackupTask constructor instead of null

Recent changes to PerformBackupTask made it expect empty list rather
than null for the ArrayList<String> pendingFullBackups argument.

Bug: 35420596
Test: manual
Change-Id: I8e1f69e78856b9bb8fc8aa9ab1adcac634314a36

Merge "Save TaskRecord before calling performClearTaskLocked."

Merge "Remove mTask from AppWindowToken."

Merge "Add more instrumentation to BackupManagerService"

Merge "Move value for the Build.CUR_DEVELOPMENT constant into libcore" am: 3e6c189c68 am: 50351bd85c
am: e8903c666a

Change-Id: I45e9a8b2b084a892f754f27a044d70318eb99772

Merge "Move value for the Build.CUR_DEVELOPMENT constant into libcore" am: 3e6c189c68
am: 50351bd85c

Change-Id: I155559d6c283e58c14995cec8423924ef14bd5b0

Merge "Move value for the Build.CUR_DEVELOPMENT constant into libcore"
am: 3e6c189c68

Change-Id: I981b746a28cda09d7a75fa79faeacb2cc58b1fc4

Merge "Move value for the Build.CUR_DEVELOPMENT constant into libcore"

Merge "fd_utils: switch to libbase logging." am: 982f42f71c am: a31b7d383f
am: 5d763a36b4

Change-Id: I837fa62065df0acd7f3fc8d3e605709861107ab4

Merge "fd_utils: switch to libbase logging." am: 982f42f71c
am: a31b7d383f

Change-Id: I55defd68178a4374e0f1b138fa320db1cfe8a448

Merge "fd_utils: switch to libbase logging."
am: 982f42f71c

Change-Id: I22364e3242317761ac27c51c54efb84dd6aee815

Merge "fd_utils: switch to libbase logging."

Merge "fd_utils: add missing logging for a couple of failure cases." am: 9dbcff7d73 am: 04d17c07cc
am: 7e0db82cd5

Change-Id: Ibbcdcb8d4b9e47ef7963e1521ceb9d4179de13de

Merge "fd_utils: add missing logging for a couple of failure cases." am: 9dbcff7d73
am: 04d17c07cc

Change-Id: I736ac35b6f040b9470ec51fb7804bebb38afb6fe

Merge "fd_utils: add missing logging for a couple of failure cases."
am: 9dbcff7d73

Change-Id: Iabd49e8f322e926b1b157ff85f1e9e4f66b0f56a

Merge "fd_utils: add missing logging for a couple of failure cases."

Merge "Use iterator to remove elements from collection when traversing it"

Merge "Fill-provider can control the negative button label and listener"

Merge "Allow auto-fill services to disable themselves"

Merge changes from topic 'secondary-dex1' am: d56d98b860 am: e6f3ed6aef
am: 969327b823  -s ours

Change-Id: I03c3f76927e4d9e46d512fe9074cacb10d743167

Merge "Add a missing write lock to settings state."

Merge changes from topic 'secondary-dex1' am: d56d98b860
am: e6f3ed6aef

Change-Id: Ia637e1c6caccf33137cde413779911dab5677147

Merge "Remove the name length limit for system properties." am: c657b769ab am: 6f00e6e977
am: b088dd5d15  -s ours

Change-Id: I4c40815a0bbb4bf216fb9ac98c9c37853a74c5a0

Merge changes from topic 'secondary-dex1'
am: d56d98b860

Change-Id: Ia9c1329584597a6f30aeaa2bd8e8c979705d8eea

Merge changes from topic 'secondary-dex' am: 9e88c53337 am: a400ccdaa0
am: aa95700b54

Change-Id: I79aa3a24a3982ca60bb1b51323e0b9f6736a7119

Save package dex usage info after secondary dex reconciliation am: 61fd6eab46 am: ade90057c6
am: efb30d85a5  -s ours

Change-Id: Iff414ec5fcaa700a84c92067425e3717bfddc7b6

Compile secondary dex files during background dexopt job am: 574994afde am: 112a61bf32
am: 36bc489096  -s ours

Change-Id: I5b884dd7119d79489c303c3e5c6c1b1871a212b2

Add a shell command to force the background dexopt job am: f7edab63d9 am: 90ac28d618
am: 6f80bb4166  -s ours

Change-Id: I65e4ec53d906309914bf8e1b3420c2264c8f7370

Merge changes from topic 'secondary-dex1'

* changes:
  Use DexManager logic to decide if a package is used by other apps
  Skip class path checking for secondary dex files

A bit more refactoring in BackgroundDexOptService am: a57ef16375 am: a0358c9e32
am: ff7bde0fa3  -s ours

Change-Id: I58955e558e574e01beaf40e5ecde994e79a3a2d4

[PM] Clean up logic for secondary dex oat files am: e69fba3f7f am: 110e897eaa
am: 29f08b9f1b  -s ours

Change-Id: I9d1626a88d2d3743f2ca2c2493c8a46d3cbaf80c

Compile secondary dex files in DexManager am: 7218363a92 am: 468be64cf2
am: 97b4de24d2  -s ours

Change-Id: I243bf467552438f5085b609a23d1a6161e28506b

Notify DexManager about new package installs am: fcd672fca3 am: 27ea592098
am: a6adb97aa0  -s ours

Change-Id: Iaad413350849a6e1fc225d26d50977f5a4983f0f

Add a missing write lock to settings state.

Atomic file requires sync between writers otherwise we may end
up with partially written settings file and no backup to recover.

Test: not testable by how we hold a mutex

bug:35915719

Change-Id: I97eebf869fa7e4989dcd2a29e4418c22706edcb8

Fill-provider can control the negative button label and listener

Test: New CTS tests added and all autp-fill tests pass

bug:35869744

Change-Id: I55e5938038e612d728498658a763208f7022f85a

Allow auto-fill services to disable themselves

1. Added a new API for a connected auto-fill service to
   disable itself

2. Added a new shell command to destroy all pending sessions
   which is used in CTS tests

3. Fixed a bug where the unbind timeout was in minutes
   instead of seconds

Test: wrote CTS tests, all auto-fill tests pass

bug:35848030

Change-Id: I681605aa0b8c004a0f14e30b57117c291d89a894

Merge "Make sure to call the original reply-to receiver when..."

Merge "Resize task to fullscreen when moving from pinned stack."

Use DexManager logic to decide if a package is used by other apps

Stop relying on the file marker recorded by the runtime. We have the
same data available from the DexManager.

The clean up CL to remove the handling of the file markers will follow.

Test: verify that "shared packages" (e.g. gmscore) are not compiled with speed-profile.

Bug: 32871170

(cherry picked from commit 07b6eabe79261267ecd7114790e96e1f6828672a)

Change-Id: I2969b75fa77d38dde0073ef67aa89b1dc91237b5

Merged-In: I6cc5834ac88489e69896cd5fba9ed9968aa7f5a2

Skip class path checking for secondary dex files

Pass '&' marker to dex2oat classpath when compiling secondary dex files.
This will skip class path checking when loading the oat files.

Test: adb shell cmd package compile -m speed -f --secondary-dex com.google.android.gms
      oatdump --header-only --oat-
file=/data/user/0/com.google.android.gms/app_chimera/m/00000006/oat/arm64/DynamiteModulesC_GmsCore_prodmnc_alldpi_release
      check that the class path contains the special '&' marker

Bug: 32871170

(cherry picked from commit aae35767b6bca6f872c906a0fd38d7adb61217bd)

Change-Id: I7f3c6743e749316ee02e8586a525ad28ae1ef765

Merged-In: If0628aeb4e3f5717604bfc4a87a4b1d438e5fa65

Merge "Fix issue #35365651: ActivityManager.getPackageImportance() doesn't..."

Merge "Revert "Modify SurfaceView to use SurfaceFlinger child surfaces.""

Merge "Revert "Correct dissapearing SurfaceView during exit animations.""

Merge "Revert "SurfaceView: Include surfaceInsets in position calculation""

Revert "Correct dissapearing SurfaceView during exit animations."

This reverts commit 026a6c45e77d3dec8f7ae26b74fe6fba18021caf.

Change-Id: I3ff0d64ca9f91aef4b5a53ed47e4d05f0d128e11

Revert "SurfaceView: Include surfaceInsets in position calculation"

This reverts commit 8c31e1db2a2cea4278503d6b9b27bca87946ef36.

Change-Id: Id25f493ca272878c2105f9cd302db1e05284998d

Make sure to call the original reply-to receiver when...

replacing a queued broadcast.

- Also don't replace a broadcast for a different user.

Test: Manual test with the following test code:

Intent intent = new Intent(Intent.ACTION_PROVIDER_CHANGED)
        .addFlags(Intent.FLAG_RECEIVER_REPLACE_PENDING);
AlarmManager alm = this.getSystemService(AlarmManager.class);
long time = SystemClock.elapsedRealtime() + 5 * 1000;
for (int i = 0; i < 5; i++) {
    alm.setExact(AlarmManager.ELAPSED_REALTIME, time,
            PendingIntent.getBroadcast(this, i, intent, PendingIntent.FLAG_UPDATE_CURRENT));
}

Without this CL, after the alarm fires, AlarmManagerService.mBroadcastRefCount
is left > 0 and the wake lock is held forever.

With this CL, mBroadcastRefCount eventually gets back to 0.

Bug: 35779096
Change-Id: I4e21c94b08f25f9ca1242182670ff4a69f8bd9f2

Revert "Modify SurfaceView to use SurfaceFlinger child surfaces."

This reverts commit 693f3432ae77d1fcfaaf9d168de861192aacb4c4.

P0: When playing encrypted content the Fugu displays a blank screen.

Test: with topic "surfaceview-without-wm" reverted, encrypted playback
works on ToT oc-release. See repro steps in 35917840#12.

bug:35917840

Change-Id: I37fa1e427daff3a1c18ed1c92d035421d891f67c

Merge "Remove the name length limit for system properties." am: c657b769ab
am: 6f00e6e977

Change-Id: If2daf92e3b7aed37832aa0787a2c689d8bdedd97

Merge "Remove the name length limit for system properties."
am: c657b769ab

Change-Id: I700b379c0482d7d9609183a39a5c9664746ea5aa

Merge changes from topic 'secondary-dex' am: 9e88c53337
am: a400ccdaa0

Change-Id: I95ed98e02fe5a54699814eb617697852095ca55d

Save package dex usage info after secondary dex reconciliation am: 61fd6eab46
am: ade90057c6

Change-Id: I094a69564b25f84ed36b51b9064c4bb499088802

Compile secondary dex files during background dexopt job am: 574994afde
am: 112a61bf32

Change-Id: I2fd8f38ad11b9ba8e4eeff21b76356eb3c66a762

Add a shell command to force the background dexopt job am: f7edab63d9
am: 90ac28d618

Change-Id: I3fef4ed490036753841e09760ec929c24da0a198

A bit more refactoring in BackgroundDexOptService am: a57ef16375
am: a0358c9e32

Change-Id: Ie1e511761121f13bf106dd888129d5e3b168b818

[PM] Clean up logic for secondary dex oat files am: e69fba3f7f
am: 110e897eaa

Change-Id: I83d161595df4d1fee18652b8432a1950fb6f367d

Compile secondary dex files in DexManager am: 7218363a92
am: 468be64cf2

Change-Id: Ie0ebc07e2cf4fd4892b9171e1823d63414eef194

Notify DexManager about new package installs am: fcd672fca3
am: 27ea592098

Change-Id: I3a71f8d4a6fcdad25b0b90a800d109bdeb16b3eb

Merge "Remove the name length limit for system properties."

Merge changes from topic 'secondary-dex'
am: 9e88c53337

Change-Id: I77e80eaf9274a64348c80b251e865ecdb6e53dbc

Save package dex usage info after secondary dex reconciliation
am: 61fd6eab46

Change-Id: I90a95645e9d2acf1b2377e165a64b34b1a678295

Compile secondary dex files during background dexopt job
am: 574994afde

Change-Id: I204983f6221c07467b450ea9e0bf71b6d15926e1

Add a shell command to force the background dexopt job
am: f7edab63d9

Change-Id: Ie77bc356a9ff4786c23a06d1267dc7ca5e45a90a

A bit more refactoring in BackgroundDexOptService
am: a57ef16375

Change-Id: Ic258b5484eedcc68650b474acb4fa3eae8589c82

[PM] Clean up logic for secondary dex oat files
am: e69fba3f7f

Change-Id: I223b27221acf25e628003dde0f63cffe66779753

Compile secondary dex files in DexManager
am: 7218363a92

Change-Id: I53e7864b097cf9cff702e9185bb1c9091d97fe63

Notify DexManager about new package installs
am: fcd672fca3

Change-Id: I63514deebef293d52f7811023b0915cf655514f6

Merge "Revert "Compute merged configuration changes""

Merge changes from topic 'secondary-dex'

* changes:
  Fix typo in dexopt.secondary system property
  Save package dex usage info after secondary dex reconciliation
  Compile secondary dex files during background dexopt job
  Add a shell command to force the background dexopt job
  A bit more refactoring in BackgroundDexOptService
  [PM] Clean up logic for secondary dex oat files
  Compile secondary dex files in DexManager
  Notify DexManager about new package installs

Revert "Compute merged configuration changes"

This reverts commit 0d8ff25fabae95d5ffc826adbb68b4036d41af41.

Reason for revert: Original CL caused b/35911519

Change-Id: Ice82b04dd00068d10f88c0d248bf95877e12cda3

Save TaskRecord before calling performClearTaskLocked.

If the activity exists in a task that is in persistent storage, the
task reference will be cleared when its brought to the foreground in
this method. Alternatively, if the activity is finished immediately
as a result of clearing task, the task reference will be lost.

Therefore, we must persist the task reference before calling it.

Change-Id: Id38cf6654f5d4f5b727441048dc6b9a7568b574e
Fixes: 35868563
Test: manual (code and bugreport inspection)

Merge "Fix potential use-after-free in LayerUpdateQueue"

Merge "Javadoc for the recently added getMetrics()"

Merge "Update SkiaDisplayList to use the modified SkLiteDL"

Merge "hotspot2: expose string constants for cert type and auth methods" am: 603fb2ef56 am: d919593b9d
am: 19df6d208e

Change-Id: I184f1a40a9dc854071915609dc13322d0fc86a97

Merge "hotspot2: expose string constants for cert type and auth methods" am: 603fb2ef56
am: d919593b9d

Change-Id: I04651e913216903316e8625672cf62609e76058d

Merge "hotspot2: expose string constants for cert type and auth methods"
am: 603fb2ef56

Change-Id: I6ae4811e97a99c04a423c1390d87a8c21f4ae416

Merge "Fix accessibility description of battery saver in QS"

Remove the name length limit for system properties.

Bug: http://b/33926793
Test: builds
Change-Id: Ib89b66b7abebc9a28961a0c9032b99947e3db0d6
Merged-In: Ib89b66b7abebc9a28961a0c9032b99947e3db0d6

Merge "Remove the name length limit for system properties."

Merge "hotspot2: expose string constants for cert type and auth methods"

Merge "Update package use info when the app data is updated"

Fix potential use-after-free in LayerUpdateQueue

Change-Id: I090af2191576175b165a9db574a80123c16f0778
Fixes: 26548204
Test: Builds & hwui unit tests pass

Merge "Let ResourcesManager generate CompatResources"

Merge "Add API to indicate which tile was long-clicked."

Fix typo in dexopt.secondary system property

Test: adb shell cmd package bg-dexopt-job
Bug: 32871170

(cherry picked from commit 4466c5ad84e4429284b7d5e13ad8a87c924ea7fc)

Change-Id: Ibfe3111198877ed0d758b68be89afb23c4756053

Merged-In: Ic8a14a3fc5c9aa63f6cb376f337b5bf459997325

Save package dex usage info after secondary dex reconciliation

Bug: 1833991
Test: add a bogus entry to /syste/data/package-dex-usage.list
      adb shell cmd package reconcile-secondary-dex-files
com.android.google.gms
      wait a bit and check the bogus entry is gone

(cherry picked from commit b1097411028103b6c88ce325af23d2ff1ec746c8)

Change-Id: I23248b11ea07bceb6527f55bf62618d1b2c3d83d

Merged-In: Ic07126bfb8730933081a5e010e3c357d338786e8

Compile secondary dex files during background dexopt job

Extend the backgound dexopt job to compile secondary dex files when
"dalvik.vm.deopt.secondary" system property is set to true.

Test: adb shell cmd package bg-dexopt-job
Bug: 32871170

(cherry picked from commit 51f521c3bf46e6040f36757bc53ea57ddc7be85e)

Change-Id: I220ff4ea26c4eee030ab1a9f4c957a09aa9f2f4c

Merged-In: If3a4b77f785e92c7f2f7869e25ff9b99904f165a

Add a shell command to force the background dexopt job

Add 'bg-dexopt-job' command to 'adb shell cmd package' which will force
the background  dexopt job to run the optimizations immediately.

Note that the command only runs the background optimizer logic. It may
overlap with the actual job but the job scheduler will not be able to
cancel it. It will also run even if the device is not in the idle
maintenance mode.

Test: adb shell cmd package bg-dexopt-job
Bug: 32871170

(cherry picked from commit cb5f41ea11b1a6fcd0977a64ee146dde8f537076)

Change-Id: I82c781d46fe16e7b7566a5b5ee91be723a2125cb

Merged-In: I8b0112074220b2e09baf13d842401ee1cf306a89

A bit more refactoring in BackgroundDexOptService

Extract convenient methods to help with:
- adding a shell command to run the background dexopt job
- extending idleOptimizations to cover secondary dex files

Test: device boots, background job run.
Bug: 32871170

(cherry picked from commit c660475aafe91269be306c802823cf543005ce36)

Change-Id: I146e4faeea90f2d58144a5dba26ae7f90a36b402

Merged-In: I18fde62a1cb05c030a620722d5f217589beaefb1

[PM] Clean up logic for secondary dex oat files

Add logic in DexManager to reconcile secondary dex records with the
actual files on disk. If secondary dex files are moved or removed then
DexManager will remove the generated oat files during the call to
reconcileSecondaryOdex() and update its internal state.

Add 'adb shell cmd package reconcile-secondary-dex packageName' which
will force DexManager to sync its data with the actual secondary dex
files.

Test: devices bots
      runtest -x .../PackageDexUsageTests.java
      runtest -x .../DexManagerTests.java
      adb shell cmd package reconcile-secondary-dex
com.android.google.gms (after artificially/temporarily renaming some
dex files)

Bug: 32871170

(cherry picked from commit c22c30ed1c05c5c24185dc4d380d1c5026923d46)

Change-Id: Ied9fcbfe367ed3a8250a9ba8d202518b264c64e8

Merged-In: Id2d72dc89995f89cf1ddf79ae4e992afd3f3c127

Compile secondary dex files in DexManager

Add logic for secondary dex compilation in DexManager. Also, extend the
`cmd package compile` command with '--secondary-dex' option which will
compile all used secondary dex files for the given package.

Test: flash & boot,
      cmd package compile --secondary-dex -f -m speed
com.google.android.gms
Bug: 32871170

(cherry picked from commit c22c30ed1c05c5c24185dc4d380d1c5026923d46)

Change-Id: I3cf26edbfb5e9f3683e608bb2a596e1ab5682bd4

Merged-In: Ia2b95cb70d26c4ead5cb650047641b0881ca84ae

Notify DexManager about new package installs

This will update the internal cache so that the secondary dex files of
the new package can be found when loaded.

Test: runtest -x .../DexManagerTests.java

Bug: 32871170

(cherry picked from commit 0d4b8f8b0c963d9a1f5cb6aff11a11195a3df225)

Change-Id: I34008f6d262f19e88a346a36b83e4093b4129105

Merged-In: I0cb7158af249feeb01cf578a7544dda978eede9d

Merge "Add enums for battery settings page."

Merge "New design for instant apps in app details header"

Merge "Fix NavGesture version"

Merge "Fix emergency only edge case"

Merge "Use DexManager logic to decide if a package is used by other apps"

Merge "SurfaceView: Include surfaceInsets in position calculation"

Fix accessibility description of battery saver in QS

Change-Id: I412a4f15c8a7c77a087f63d9331204d72e1dcb98
Fixes: 35617900
Test: enable accessibility, select battery saver