Renames nanoapp instance IDs to handles in ContextHubService

Use handles to refer to nanoapp instances at the service to be
consistent with the terminology at NanoAppInstanceInfo, and to
reduce confusion with other instance IDs (e.g. ones maintained
in the CHRE implementation).

Bug: 67734082
Test: None
Change-Id: Iab60b4df20383907668824ac95c3f5a1a559ee6b

Creates a NanoAppStateManager at ContextHubService

Created to decouple the internal nanoapp state management at the
service with ContextHubService class

Bug: 67734082
Test: Compile and flash, run CHQTS and verify pass, also verify with
logs that the cache is updated accordingly after load/unload/query
Change-Id: Ie25460e7f8d903cdcb22f0a1c4870bb93ccbad3e

Merge "Enable fast drawing for solid color nine patch rectangles"

Merge changes I2a808b9f,I8cbd4651

* changes:
  Implements queryNanoApps API
  Implements the new loadNanoApp/unloadNanoApp APIs

Merge "Simplify IMMS#startVrInputMethodNoCheck a bit"

Merge "Fix issue #69820174: The size of /proc/meminfo is larger than its stored buffer."

Merge "Restore volume expansion."

Merge "Fixed button bar alignment issue in RTL languages" am: 403c8f72e3 am: 2063b1f792
am: bf62defed2

Change-Id: I7113cb185049848344c01b857e8a8646e9967432

Merge "Fixed button bar alignment issue in RTL languages" am: 403c8f72e3
am: 2063b1f792

Change-Id: I7493a720ef88f0b40ebed483d474a34ad9ee5da4

Merge "Fix resizing when hardware renderer is off."

Merge "Fixed button bar alignment issue in RTL languages"
am: 403c8f72e3

Change-Id: I8a14f71b20fd07ac3fd9025ba4e303a11709fe0a

Merge "Fixed button bar alignment issue in RTL languages"

Implements queryNanoApps API

Bug: 67734082
Test: Run a test app to use the new query API, verify returns the
expected set of nanoapps
Change-Id: I2a808b9f80dd0d9168466f80b79ca1fa35da242c

Merge "Fix system crash in certain jobs-removed cases"

Enable fast drawing for solid color nine patch rectangles

Pass a hint to Skia, about which lattice rectangles are solid
color rectangles.

Bug: 69796044
Test: Measured ninepatch performance using sample app from the bug
Change-Id: Ib07b1b64c78ab16195f9af88a989d28682084565

Restore volume expansion.

Test: manual
Bug: 63096355
Change-Id: Ibff2826084353b66a6e1376cb9dda3e01dcf0e72

Merge "Update GrContext creation calls to use sk_sp"

Merge "Fixed a crash with the top headsup entry"

Merge "Add auto-doc support for @StringDef."

Merge "[DeadLock]AMS PMS MountService" am: 998ac99db4 am: 41fb54b955
am: 81544d2a71

Change-Id: Ib92a19bce9fd9086c302c2cf4606b5d10161fe85

Merge "[DeadLock]AMS PMS MountService" am: 998ac99db4
am: 41fb54b955

Change-Id: I15056a18d70a1d17c52799f71e8faf5c565bb8aa

Merge "[DeadLock]AMS PMS MountService"
am: 998ac99db4

Change-Id: I5910f42d91c3c3edcc092a87cc2e291d6e83e75e

Merge "[DeadLock]AMS PMS MountService"

Merge "Keystore: Fix KeyGenParameterSpec parceling"

Merge "Aapt2 convert: skip processing resources when resource table is absent. Test: manual"

Update GrContext creation calls to use sk_sp

Test: manual testing

Change-Id: I68d8706ffa7bc7c6622ea0b8c45d7297a131f6bf

Merge "Tidy up changes for timezone update code"

Keystore: Fix KeyGenParameterSpec parceling

Fix the way KeyGenParameterSpec is parceled, by correctly handling
default and null values for some of the fields.

A recent CL added the ability to parcel/unparcel KeyGenParameterSpec (by
a separate class).
Due to refactoring late in the CL review cycle, the parceling code did
not take into account a few edge cases.

Unit tests:
m -j KeystoreTests && adb install -r out/target/product/marlin/data/app/KeystoreTests/KeystoreTests.apk
adb shell am instrument 'android.security.tests/android.support.test.runner.AndroidJUnitRunner'

CTS tests:
cts-tradefed run commandAndExit cts-dev -a armeabi-v7a -m CtsDevicePolicyManagerTestCases -t com.android.cts.devicepolicy.DeviceOwnerTest#testKeyManagement -l DEBUG

Bug: 69337278
Test: Keystore unit tets (see instructions above) and cts Key Management test.
Change-Id: Ie08f42b07fb55b6fa1d8fb73c89d69687c97e214

Aapt2 convert: skip processing resources when resource table is absent.
Test: manual

Change-Id: Iafe20f98857a29d625e7a57a9201db6279d43e45

Merge "API Review: Drop "Button" in is/setLogoutEnabled method"

Merge "Add TaskPositioningController"

Simplify IMMS#startVrInputMethodNoCheck a bit

This is a follow up CL to a recent CL [1] that added VR-IME support.

Currently, IMMS#startVrInputMethodNoCheck() internally calls
IMMS#setInputMethodEnabled(), which enforces the caller user ID check
and caller permission check for WRITE_SECURE_SETTINGS.

This appears to make sense for IMMS#startVrInputMethodNoCheck(), but
in reallity IMMS#startVrInputMethodNoCheck() always gets called on
non-binder system server thread hence those checks will never fail.

Note that such a call path is fine and not a security issue.  What this
CL tries to address is just replace IMMS#setInputMethodEnabledLocked()
to make it clear that IMMS allows such an operation for callers that
have RESTRICTED_VR_ACCESS is intentional.

To summarize, this CL just removes redundant operations and there
should be no visible behavior change.

 [1]: I1db7981b5198e7e203d4578cae7e5b6d20037d0d
      89a6c48a8b2e54f9b93211c20a126edab0eefe35

Bug: 63037786
Test: compile
Change-Id: Iec43177bd698de6efa453d64ceafec3164538151

Add TaskPositioningController

The CL creates new TaskPositioningController class which lives over the
process WindowManagerService's life time and handle task positioning
tasks like resizing or moving by drag.

Bug: 64149458
Test: atest com.android.server.wm.TaskPositioningControllerTests
Change-Id: I65b3226ce5fa93c16037671ab9cd3b3d84d8f274

Merge "DefaultNetworkEvent: fix two bugs with validation times" am: c0d951c3e0 am: 144d9e9f83
am: 51e563aefb

Change-Id: Ide65494bfe095533a06c4b33002fbd260306f8a0

Merge "DefaultNetworkEvent: fix two bugs with validation times" am: c0d951c3e0
am: 144d9e9f83

Change-Id: Iad5a7cda8b73f638aa7db8ae38bdccef1cbeae85

Merge "DefaultNetworkEvent: fix two bugs with validation times"
am: c0d951c3e0

Change-Id: I94489cb28fad5d8adddb200b44d72bab2be70b97

Merge "DefaultNetworkEvent: fix two bugs with validation times"

Merge "Add helpers for generating keys, wrapping them, and persisting them"

Merge "Prevent reporting fake package name - framework"

Prevent reporting fake package name - framework

Test: added AccessibilityEndToEndTest#testPackageNameCannotBeFaked
      cts-tradefed run cts -m CtsAccessibilityServiceTestCases
      cts-tradefed run cts -m CtsAccessibilityTestCases

bug:69981755

Change-Id: I13304efbee10d1affa087e9c8bc4ec237643283e

Merge "Switch @IntDef from long to int, and add @LongDef"

Merge "Lock free animations (1/2)"

Lock free animations (1/2)

First CL that introduces SurfaceAnimator/LockFreeAnimator

We start our synchronized app transition journey by showing that
the concept works by using WindowState animations as proof of
concept.

The main class in this CL are SurfaceAnimator and
SurfaceAnimatorRunner. When we start an animation on a Window, we
create a new bufferless surface, called "The Leash", in the
hierarchy and attach the surface of WindowState onto it, while
attaching the leash onto the old surface parent which is still
responsible for z-layering.

Then, we pass off the Leash into SurfaceAnimationRunner, which then
changes the surface properties of Leash in every animation frame,
without holding the WM lock. While it's doing that we can still
update the z-layering of the window, or even relayout the window
of needed - the important surfaces for this are still under WM's
control.

In case the animation is finished the window surface gets
reparented to its original parent, and the leash is abandoned.
Note that the reparenting is done in the same transaction as
processing the animation finish, such that we don't end up with
a flicker in case of a disappearing animation, where the window
surface gets destroyed.

In case the animation needs to be cancelled, WM can revoke control
of the leash by reparenting the window surface. Even if the
cancellation signal is heavily delayed, WM immediately regains
control over the surface by reparenting it within a transaction.

We also introduce the concept of animating a WindowContainer. We
clean up isAnimating:
- isLocalAnimating: is the container itself animating
- isAnimating: is the container or one of its parents animating
- isSelfOrChildAnimating: is local animating or any child
animating.

SurfaceAnimationRunner also needs it's own thread so it's not getting
bogged down by any WM lock contention by processing regular
animation frames. We call that thread android.anim.lf (lockfree).

Now, imagine that SurfaceAnimationAnimator would sit behind an IPC in
another process and instead of animating WindowState, we'd animate
AppWindowToken. Then, synchronized app transitions would be done.

Test: go/wm-smoke
Test: SurfaceAnimatorTest
Test: SurfaceAnimationRunnerTest
Test: WindowContainerTests
Bug: 64674361
Change-Id: I10d41f7a289ab2158da3f2f1c3ddd78edd1efc86
Exempt-From-Owner-Approval: Tiny change unrelated to display management.

Merge "Revert "Remove empty ScopedArray constructor."" am: f6a0a7491b am: 73fce7397e
am: d3e197a1bd

Change-Id: Idf3497ce71c0a03ed440bb35c09f5247a977f6ba

Merge "Revert "Remove empty ScopedArray constructor."" am: f6a0a7491b
am: 73fce7397e

Change-Id: I8bb88e7b91928ae8fc79233a74fe545aa3d40bb7

Merge "Revert "Remove empty ScopedArray constructor.""
am: f6a0a7491b

Change-Id: I9c49b4209e98e90bea62ad1656f7405d771395c9

Merge "Revert "Remove empty ScopedArray constructor.""

Revert "Remove empty ScopedArray constructor."

Libcore tests still fail.

This reverts commit cee368b2b8299db5808284a30e2aa14d0aff8b4a.

Change-Id: I4bcdb7bf08d09d563e54cd8fe005402ae3579164

Switch @IntDef from long to int, and add @LongDef

This CL mirrors changes made to the android.support.annotation version
of IntDef, to keep the usage and semantics identical (though the
internal version of @IntDef and @LongDef are of course hidden and not
part of the SDK.)

Test: These annotations have source retention and therefore have
no runtime impact; the change was compiling the SDK.

Change-Id: Idaf47e8d983c88be1bd8f938615c86611014b45a

Merge "Convert legacy-test, test-runner, and test-mock to Android.bp"

DefaultNetworkEvent: fix two bugs with validation times

This patch fixes two bugs around the validation time calculation for
default network metrics:
  - for events representing the absence of a default network, the
    validation time was not correctly set to 0.
  - for events representing a default network, the validation time was
    always set to the duration time. This was overlooked by commit
    380a0638588d.

Also fix a minor printing padding issue in DefaultNetworkEvent#toString

Test: manually inspected the output of $ adb shell dumpsys connmetrics
      while enabling and disabling wifi.

Change-Id: I0eb4ccdf7a61d3097d0661104cb40d738e59772a

Merge "BootReceiver: Added an event log after the dropbox file copy is successful." am: 5323d42b82 am: c044749416
am: 89df84bfb4  -s ours

Change-Id: Ic5007c1e5bf5153691d3d1cc743c449f7f583bc7

Merge "BootReceiver: Added an event log after the dropbox file copy is successful." am: 5323d42b82
am: c044749416

Change-Id: If096fea07e31abbd8c5c418c776296c1200009d9

Merge "BootReceiver: Added an event log after the dropbox file copy is successful."
am: 5323d42b82

Change-Id: Iaf53e8a99b61d68c3b920b76c78e9208845c56f3

Merge "BootReceiver: Added an event log after the dropbox file copy is successful."

Merge "Revert "Send empty LinkProperties when entering StoppedState."" am: e339662247 am: 66c8516600
am: a312cf38f9

Change-Id: Ib7aff28ca5088bf5ebbc86ab7ee5c607e35d6a2b

Fix system crash in certain jobs-removed cases

Null happens.  Check for it before calling size().

Fixes: 69630116
Test: atest CtsJobSchedulerTestCases

Change-Id: I17ec206946a6a21ff459e2e031cface5499f5264

Merge "Revert "Send empty LinkProperties when entering StoppedState."" am: e339662247
am: 66c8516600

Change-Id: I9c7175d9e2db45780f14630db43a6ab5672dfe3d

Merge "Improved AccessibilityNodeInfo.toString()."

Merge "Revert "Send empty LinkProperties when entering StoppedState.""
am: e339662247

Change-Id: I402078a9ad6d92441cf34ca0b9459ee51adac984

Merge "Move IME related API methods from IMM to IMS."

Merge "Revert "Send empty LinkProperties when entering StoppedState.""

Merge "PROTECTION_FLAG_VENDOR_PRIVILEGED as @TestApi"

Fix issue #69820174: The size of /proc/meminfo is larger than its stored buffer.

Make it larger.

Bug: 69820174
Test: booted and ran.
Change-Id: Ic0d7db54e9f34b11f81b3b4fcd8cac15753a548c

Merge "Merge "[AWARE] Make PeerHandle comparable/hashable" am: 8d621cc042 am: bd82991016  -s ours am: 8b4960ec91  -s ours"

Merge "Use stable rect to determine if we should adjust for IME in split-screen"

Merge "[AWARE] Make PeerHandle comparable/hashable" am: 8d621cc042 am: bd82991016  -s ours
am: 8b4960ec91  -s ours

Change-Id: Ic53447dfd6cf026374f9fb9471db8d9d0563ae65

Merge "Add default value for battery app list flag"

Add auto-doc support for @StringDef.

Behaves pretty much the same as @IntDef, but now supports "suffix"
in addition to "prefix" when matching constants.

Test: manual docs output looks sane
Bug: 70406696
Change-Id: I35064b0f9f36f1f13ccdb40302d818a004014f15

Use stable rect to determine if we should adjust for IME in split-screen

Regression was introduced in ag/3174240 where the content rect used to
be cached before the layout IME so the IME didn't affect the content
rect. However, we don't want content rect to mean something differect
here as content rect excludes the IME space in the rest of the system.
So, changed the one use case we have for this to use stable rect.

Change-Id: I13e23baf87dbcaf21a8e0624f73b9af6116f7262
Fixes: 69425766
Test: Steps from bug

Merge "[AWARE] Make PeerHandle comparable/hashable" am: 8d621cc042
am: bd82991016  -s ours

Change-Id: I0d878f4ae7067e875879e683525d20338f2e3405

Merge "Adding cache of IMS provisioned values."

Merge "Post requestFocus so that it happens after layout"

Merge "[AWARE] Make PeerHandle comparable/hashable"
am: 8d621cc042

Change-Id: I31a1ebcecd49b620462f2fb917e6c0c0d8f4d06e

Merge "[AWARE] Make PeerHandle comparable/hashable"

Merge "Another attempt at diffs in offline SDK docs."

Merge "Timeout app predictions after 12 hours"

Merge "Turn on PGO for hwui"

Merge "[RTT2] Update RTT service name"

Merge "Bluetooth: Fix boolean logic in quiet mode enable" am: 1801711c53 am: b9d49781ac
am: 91c8a6edc1

Change-Id: I7a584f49369f929de68cd1b1aa25042f000f1522

BootReceiver: Added an event log after the dropbox file copy is successful.

The log can be used to test if LAST KMSG or other items are copied
to dropbox successfully, especially in user builds without root
privilege.

BUG: 69685635

Test: manually verified the desired log from bugreport on user
and userdebug builds.

Change-Id: I6570d95538d678c98d261690ca3c20416d7a31c6
Merged-In: Ie6033bf04c7f79fc596761ab751aa5fcea2c1130
(cherry-picked from commit bafcd7b59519ce67738c870cf9b69a7c0583cc9b)

Merge "Define fields in incident header and add a new proto to log internal incidentd metadata"

Merge "Bluetooth: Fix boolean logic in quiet mode enable" am: 1801711c53
am: b9d49781ac

Change-Id: I8173afb73788a3de9ce7b1af07e653f9b76fbefa

Merge "Bluetooth: Fix boolean logic in quiet mode enable"
am: 1801711c53

Change-Id: I58d0f361db35998b30c70cfd380e35b63e543524

Revert "Send empty LinkProperties when entering StoppedState."

This reverts commit b3f9f4a30502477e71fc93037dc58080fddb1853.

Reason for revert: Crash in WifiStateMachine (b/70394432)

Change-Id: I3a96c822eddcebcdd6621cee878b865171d094e8

Post requestFocus so that it happens after layout

Otherwise, the ScrollView remains focused (due to another bug
that can't be fixed until RV is fixed) and causes the tests
to fail

Bug: 70384906
Test: android.widget.scroll.arrowscroll
Change-Id: I9691a38d46a8c430fbd360116aa0ff37c34a299b

Merge "Bluetooth: Fix boolean logic in quiet mode enable"

Merge "Changes need to implement heavy-weight gaming app tests."

Merge "Fix issue where brightness mirror listener wouldn't be released"

Improved AccessibilityNodeInfo.toString().

Test: manual verification
Bug: 70292857

Change-Id: I2d81071402c429ed29e6c5b443956b085944e12f

Merge "Age jobs into runnability even in infrequent buckets"

Another attempt at diffs in offline SDK docs.

We already generate offline SDK docs as part of "docs" build, and
this change simply adds the "-diff" targets so they're also included.

Test: make -j48 docs dist
Bug: 69115817
Change-Id: Iecd3f070280b078fbdca91fdc1a6b8d65b8df863

Merge "Fix issue #23820845: Update AlarmManager.setAlarmClock() documentation..."

Timeout app predictions after 12 hours

Ignore predicted bucket if it's been too long since the
last update, and use internal timeout logic to set
standby buckets.

Keep track of last predicted time and persist it.

Bug: 70219058
Test: atest FrameworksServicesTests:AppStandbyControllerTests

Change-Id: Ib2992201d204b92fdd4a0a00754d81e7bbfc292f

Merge "BootReceiver: Added an event log after the dropbox file copy is successful."

Merge "Added test cases for DataCallResponse parcel read/write"

Merge "Add Wifi Wake notification ids to system messages"

Convert legacy-test, test-runner, and test-mock to Android.bp

See build/soong/README.md for more information.

Test: m checkbuild
Change-Id: I110f752d2fa94c44581f20d4f308d9d429da0517

Merge "Convert services to Android.bp"

Merge "Recycle ActivityStarter instances."