[automerger skipped] Merge "Merge "Use the malloc debug heap dumper." am: 0ec0c17596 am: 84c9e9a787" into pi-dev-plus-aosp
am: da5fb6eac7  -s ours

Change-Id: If0cfb06266b5d75dc50d900117a154b94823853b

[automerger skipped] Merge "Use the malloc debug heap dumper." am: 0ec0c17596 am: 84c9e9a787
am: 4d5ba2e496  -s ours

Change-Id: I4f1d4939e3b45b57dea8b8af2392501351e2b84b

Merge "Skip null values in Utils.safeForeach" into pi-dev am: c5901f5a06
am: e0d3e1c2d5

Change-Id: I9d1d3756c508771551b727755a1cab86e2aa4afb

Merge "Merge "Don't back up Assist Gesture setup/sensitivity" into pi-dev am: a74fbef8ce" into pi-dev-plus-aosp
am: fe611a5700

Change-Id: I1aba793c0c71573e06d52f14aec823a4f6d3ad93

Merge "Merge "Use the malloc debug heap dumper." am: 0ec0c17596 am: 84c9e9a787" into pi-dev-plus-aosp

Merge "Use the malloc debug heap dumper." am: 0ec0c17596
am: 84c9e9a787

Change-Id: I96c46291cc73c021945c370c7bb33164bc0a3a0b

Merge "Fix ManifestMerger issues" am: 684b5f038b am: c5e5b64b04
am: 6c40e4745d

Change-Id: Ide97a39158b8ec9310ded188fa9f75c6795fc2b5

Merge "Skip null values in Utils.safeForeach" into pi-dev
am: c5901f5a06

Change-Id: I60c1ee2c2036b883a456d916e26576bb206a39ec

Merge "Merge "Don't back up Assist Gesture setup/sensitivity" into pi-dev am: a74fbef8ce" into pi-dev-plus-aosp

Merge "Don't back up Assist Gesture setup/sensitivity" into pi-dev
am: a74fbef8ce

Change-Id: Ib14cc5a35991dec3623ecf2dfdb72cbbccd7e53e

Merge "Fix ManifestMerger issues" am: 684b5f038b
am: c5e5b64b04

Change-Id: I4b8e51aaec1f825e4ccf7ccbc60e61b82e05c509

Merge "Use the malloc debug heap dumper."
am: 0ec0c17596

Change-Id: I0f0d6a01a4a0d55f18e3f7b7ac7c64534d4b8f54

Merge "Skip null values in Utils.safeForeach" into pi-dev

Merge "WM: Fix seamless rotation"

Merge "Don't back up Assist Gesture setup/sensitivity" into pi-dev

Merge "[Notif] Reset falsing check for blocking helper"

Merge "Fix ManifestMerger issues"
am: 684b5f038b

Change-Id: I4a551483dd3f0093f11b557a8ee6e6a99cace256

Merge "Hiding replied notification icons from AOD" into pi-dev am: 37e96a8fa5
am: 0dac634ed0

Change-Id: I774d517aa5312c149c256c3955022fa28933cda9

[automerger skipped] Merge changes I90451c14,I9fb27a0a into pi-dev am: b91dd1d944
am: 22ffb55de6  -s ours

Change-Id: I29096c00ca57a4b872f31f3c79bf75239bef56be

WM: Fix seamless rotation

With the introduction of the surface hierarchy, the seamless rotation
behavior in WSA is no longer correct: it also applies the WindowState's
offset, which leads to that being applied twice.

Instead of doing that, we simply rotate the WSA surface within the place
that WindowState dictates now.

Finally, the location of the WindowState itself also needs to be
transformed into the new orientation.

Fixes: 109927566
Test: atest CoordinateTransformsTest
Test: atest 'WindowStateTests#testSeamlesslyRotateWindow'
Change-Id: I9fb27a0a8a2bddc6ec88a4fcce6d6ea00929fb91
(cherry picked from commit 41f7e9d10306603c7a4410938e77418f13f6e939)

Merge "Removed TODO introduced in ag/4344061"

Merge "Make oneway calls from AFM to AFMS."

Merge "Fix missing entries in TSMS dump"

Merge "Hiding replied notification icons from AOD" into pi-dev
am: 37e96a8fa5

Change-Id: I60a5aa171261b97b4f6de41515332fa93016eb8d

Merge changes I90451c14,I9fb27a0a into pi-dev
am: b91dd1d944

Change-Id: I58459846d6cb79d616b4bd9c83450dc911e5115c

Removed TODO introduced in ag/4344061

After further digging, we determined that the workaround is the
best way forward.

Change-Id: If046cdd16a22f0eec63a611eb21ce5e4c875896d
Fixes: 110159357
Test: N/A

Merge "Update user on user switched for color inversion"

Make oneway calls from AFM to AFMS.

AFM should not block the UI thread when calling AFMS; this CL is a
"low-hanging fruit" change that makes oneway call from the existing functions
that return void.

Test: mmma -j ./frameworks/base/apct-tests/perftests/autofill/ && \
  adb install -r $OUT/data/app/AutofillPerfTests/AutofillPerfTests.apk && \
  adb shell am instrument -w -e class android.view.autofill.LoginTest \
  com.android.perftests.autofill/android.support.test.runner.AndroidJUnitRunner
Test: CtsAutoFillServiceTestCases

Bug: 73536867

Change-Id: I09e84b797153cd02d3c02262994a2be1f38639ea

Merge "Use the malloc debug heap dumper."

Merge "Fixed an issue where the shadow wasn't rounded properly"

Fix missing entries in TSMS dump

This CL fixes another regression that was introduced when we enabled
multiple users to use spell checker APIs at the same time [1].

Somehow TextServicesManagerService#mOnGoingSessionRequests were not
fully dumped into dumpsys due to an unnecessary '++j' in the loop.

 [1]: I06c27ef834203a21cc445dc126602c799384527b
      06a2624049f09e1ad68f8ab74fdb525aacd5c1e5

Bug: 63041121
Test: Manually verified that 'adb shell dumpsys textservices' dumps
      all the elements in #mOnGoingSessionRequests.
Change-Id: Ia26ab1c8ca19e74aaefb8562fb06ff64c3d8d677

Merge "Fix ManifestMerger issues"

Merge "Hiding replied notification icons from AOD" into pi-dev

Merge changes I90451c14,I9fb27a0a into pi-dev

* changes:
  DO NOT MERGE WM: correctly draw the rounded corner / cutout overlay during rotation
  WM: Fix seamless rotation

[Notif] Reset falsing check for blocking helper

Currently, if you long press on the notif in lock screen and then are
interacting with the blocking helper UI (NotificationInfo), the content
automatically closes after 8 seconds even if you've hit stop/undo/etc.

Instead of letting this happen, reset the timeout.

Test: Manually that it doesn't timeout after 8s on LS after update
Bug: 110213459
Change-Id: I0169a7749b718714595bd7239375308f30bf4c43

Use the malloc debug heap dumper.

Instead of having the malloc debug heap dump code live in frameworks
code, call into malloc debug to do the dump.

Test: Ran am dumpheap -n <PID> <FILE> of a process with and without
Test: malloc debug enabled.
Change-Id: I7911a3ee7dcbc79dd11003a24e27ff99c8301d43
Merged-In: I7911a3ee7dcbc79dd11003a24e27ff99c8301d43
(cherry picked from commit 38e2c3bc9c50770aa5efce47f90fbcfcdda523de)

Merge "Use the malloc debug heap dumper."

Merge "Remove log spam in MediaRouter"

Merge changes Idad77721,I9717ac4b

* changes:
  Split internal interface activities from current AM interface (3/n)
  Cancel Pip animation if stack is already removed

Merge "audioservice: fix a2dp connection race condition" into pi-dev am: 706303541e
am: 94dce7ac8b

Change-Id: I001c3b390be7ee4e5522a5fd2599c9f34e6a7686

[automerger] Merge "audioservice: fix a2dp connection race condition" into pi-dev am: 706303541e am: c9abc289a0

Change-Id: Iabc86b907a6a430b824e876bab44b8cee192a71a

Merge "audioservice: fix a2dp connection race condition" into pi-dev
am: 706303541e

Change-Id: I7ce96cb90aad6c826bbeb7be0e80950f2a89b517

Merge "Abbreviate permission labels less aggressively" into pi-dev am: af76df93e3
am: 53427b7237

Change-Id: I20ae335fec3d0837bdbbe524b8a6576560dff948

Merge "audioservice: fix a2dp connection race condition" into pi-dev
am: 706303541e

Change-Id: Id4bbff8834f97f60c211dd3c45ce59af95fbb810

[automerger] Merge "Abbreviate permission labels less aggressively" into pi-dev am: af76df93e3 am: 3dce361445

Change-Id: Iea45f6992477e090e0f7ecaf2c4689354ad24786

Merge "Abbreviate permission labels less aggressively" into pi-dev
am: af76df93e3

Change-Id: I61a45dd4489c930f2c5b3fbcf6f7632771d6f18e

Merge "Abbreviate permission labels less aggressively" into pi-dev
am: af76df93e3

Change-Id: Ic78313f9bf162478b76ef35028659a1c93c9c58d

Merge "audioservice: fix a2dp connection race condition" into pi-dev

Remove log spam in MediaRouter

Bug: 110007356
Test: build
Change-Id: I249154eaf3c009429bb9731ca38b9e5eec271670

Fixed an issue where the shadow wasn't rounded properly

Previously the implementation required the bottom roundness
to be treated specially when it came to the outline and
the clip path but this isn't necessary anymore and was
leading to issues with the shadows.
We now always apply the clipBottomAmount to the outline and
the clipping and can remove the special case.

Test: add notification, observe rounded clipping on bottom
Test: add groups, observe normal rounded clipping
Fixes: 80431867
Change-Id: Id65d11ce48c08adc138e1177226a03bc25ef6345

Update user on user switched for color inversion

When the user switches, also update the user associated
with the listener for color inversion of the rounded
corners and display cutout.

Fixes: 110037623
Test: manually switch to secondary user, set color inversion on/off
and see cutout is the correct color (it should always be black)

Change-Id: I53dcffa374cb46c212f25d402c7e8a35da667624

Skip null values in Utils.safeForeach

Test: ZenModeControllerImplTest.java
Bug: 110209145
Change-Id: I4234be630c84444a70d8cb380ca6a692a4ff5cbc

Merge "Skip null values in Utils.safeForeach"

Merge "Abbreviate permission labels less aggressively" into pi-dev

Split internal interface activities from current AM interface (3/n)

3rd step in unifying the window hierarchy that is currently split
within AM and WM packages. We separate the the internal interface used
to communicate within system server dealing with activities and their
containers (tasks, stack, display) from the rest of AM internal
interface.

Test: Existing tests pass
Test: go/wm-smoke-auto
Bug: 80414790
Change-Id: Idad77721c1fe10621b9be5dced42a0a11f0183e5

Cancel Pip animation if stack is already removed

It is possible for the pinned stack to be removed before the system has
a chance to run the scheduled bounds animation on it. In this case we
should just cancel the animation.
Also fixed:
- Issue where we are synchronizing on the wrong object in
PinnedActivityStack.updatePictureInPictureModeForPinnedStackAnimation
- Issue where the enterPipRunnable in
ActivityTaskManagerService.enterPictureInPictureMode wasn't synchronized

Change-Id: I9717ac4b5849eafa6332fb0043310a0e8387a05f
Fixes: 110255768
Test: atest ActivityManagerActivityVisibilityTests#testHomeVisibleOnActivityTaskPinned

Skip null values in Utils.safeForeach

Test: ZenModeControllerImplTest.java
Bug: 110209145
Change-Id: I4234be630c84444a70d8cb380ca6a692a4ff5cbc

Do not announce selected state for QS tiles.

Change-Id: Ia461baa4a12818a0e91c6e1862eb9e13e232e720
Fixes: 110105146
Test: manual

Use FullRestoreEngine in adb restore.

PerformAdbRestoreTask and FullRestoreEngine has shared a lot of code,
remove the copy that was in PerformAdbRestoreTask.

Test: adb restore
Change-Id: Ib4ce8b50eff8aed4a2c0660808c05b80f378ea98

Merge "Protect unbindBackupAgent() with CONFIRM_FULL_BACKUP permission"

Adjust QuickQSPanel to fit Tablet display size

- Remove legacy standard_notification_panel_width
- Define default max columns for QQS layout
- Fix Pixcel C & Nexus 7 QQS Tiles over boundry problem

Test: atest SystemUITests
Fix: 110123402
Change-Id: I18ab3d966b3d18775d0b13d6d60d7c09485ba8ae

Merge "Build com.android.location.provider with java_sdk_library" am: aa5d8dfeb0 am: abdaa314f8
am: 07fed30c31

Change-Id: If214f52c141ef4d0f106bc3f5d2778f9b8f40b1e

Merge "Build com.android.location.provider with java_sdk_library" am: aa5d8dfeb0
am: abdaa314f8

Change-Id: If55a20564704769fc750fb6675533e9a0b8f3b6a

Merge "Make dumpHexString() tolerate null arrays." am: a917c1db32 am: 7bb253d078
am: 3e0f3ef8ad

Change-Id: I1fbea877d10b4f270fc1722ec1c14e9bd8f131a3

Merge "Make dumpHexString() tolerate null arrays." am: a917c1db32
am: 7bb253d078

Change-Id: Id58b8039510baa518d8d10b03237681a7b842d0f

Merge "Build com.android.location.provider with java_sdk_library"
am: aa5d8dfeb0

Change-Id: I53720ed4fcbe9f00f2f1f607d07efa9adaaf7bb4

Merge "Make dumpHexString() tolerate null arrays."
am: a917c1db32

Change-Id: Ib0160e232cab0a713853a65eef26f7a02d65c1f1

Merge "Build com.android.location.provider with java_sdk_library"

Merge "Refresh visible IME proc boost upon onBindingDied"

Fix solid circle show on message notification am: 1250cd1407
am: cb74f9c13d

Change-Id: I74ddf43c3da0f8f567c9ab6aef18a6cbdcfebca5

[automerger] Fix solid circle show on message notification am: 1250cd1407 am: 51e6cce63c

Change-Id: I20b8317a34eac3642ec6054f5f2332d1e001a7ab

Fix solid circle show on message notification
am: 1250cd1407

Change-Id: I9742ab1f93c797e23d4545d6659efcfa4808b03c

Fix solid circle show on message notification
am: 1250cd1407

Change-Id: I805908a107f7e082a0f5c3294a777235efa74b16

Fix solid circle show on message notification

Message app or other apps will use some format code or controll
code on SenderName to fit RTL or other design, and symbols will
produced by these code. The special code pattern not include these
code, so it will go charIcon flow.
Although these code is not visible, we should just ignore them to
get symbol strings

Change-Id: I20ef459b10ba7504ec0c997ed815cb485817d2bc
Fixes: 109746235
Test: Check notification form message app on RTL
Test: atest SystemUITests

Merge "Perform layout after removing the focused window"

Merge "Make dumpHexString() tolerate null arrays."

DO NOT MERGE WM: correctly draw the rounded corner / cutout overlay during rotation

When we freeze the screen, we really don't want the overlay to appear
on the screenshot - otherwise this will lead to it rotating with the
screen content. This means the overlay currently disappears during the
transition. We cannot just draw it over the screenshot, because it
might be in inconsistent state.

We fix this by temporarily undoing the effects of the screen rotation
transform on the overlay's window token. Then, once the window has
performed relayout and is redrawn in the new orientation, we switch to
that representation. This is mostly seamless rotation, with the
difference that we force it always, and it must also work for 180
degree rotation (which regular seamless rotation does not).

Do not merge, because we want to harmonize seamless rotation and the
newly added forced seamless rotation in Q.

Also move the rounded corner overlay from the display overlay layer
to the root of the hierarchy such that it can draw over the screen
off animation's ColorLayer.

Bug: 79112140
Test: Enable display cutout overlay, rotate phone to all orientations, ensure that emulated display cutout never flashes or disappears.
Test: atest CoordinateTransformsTest
Change-Id: I90451c14dc28daa3f90a74c3117548fead25af3f

WM: Fix seamless rotation

With the introduction of the surface hierarchy, the seamless rotation
behavior in WSA is no longer correct: it also applies the WindowState's
offset, which leads to that being applied twice.

Instead of doing that, we simply rotate the WSA surface within the place
that WindowState dictates now.

Finally, the location of the WindowState itself also needs to be
transformed into the new orientation.

Fixes: 109927566
Test: atest CoordinateTransformsTest
Test: atest 'WindowStateTests#testSeamlesslyRotateWindow'
Change-Id: I9fb27a0a8a2bddc6ec88a4fcce6d6ea00929fb91

Merge "StatsPullerManager not use singleton"

Merge "Revert "Change quickstep onboarding tip window type to"" into pi-dev am: b2b255a131
am: 09b4dbdf2a

Change-Id: I8d7cf4edc03e798e0cebb63c8f933eb58417fafb

audioservice: fix a2dp connection race condition

In case of fast (< 1 second) A2DP disconnect/connect sequence, there
is a potential race condition where the delayed disconnection message
(due to becoming noisy intent) is received after the not delayed
connection message.

- Make sure all messages related to device connection/disconnection
(including MSG_A2DP_DEVICE_CONFIG_CHANGE and MSG_BTA2DP_DOCK_TIMEOUT)
are handled by queueMsgUnderWakeLock().
- Make sure messages are processed in the same order as API calls.
- Add check for null address in makeA2dpDeviceUnavailableNow().

Bug: 109903807
Bug: 78837311
Test: repro steps in bugs. A2DP connection and playback

Change-Id: Ib81b3805f945f7206b1a60de74e9bbeeef89bdd0

Merge "Revert "Change quickstep onboarding tip window type to"" into pi-dev
am: b2b255a131

Change-Id: I585b9c7df35114b47ea0ab9af7ee0b881b195e4e

[automerger] Merge "Revert "Change quickstep onboarding tip window type to"" into pi-dev am: b2b255a131 am: 86e5aa8731

Change-Id: I46bc865fd3794d2ce8b79415b12be64ce0a46c36

Merge "Revert "Change quickstep onboarding tip window type to"" into pi-dev
am: b2b255a131

Change-Id: I7ad4536ff6d1ac5724f32e18693f8fcb14097a36

Merge "Expose new AppOps API that we had to leave hidden in P."

Merge "Revert "Change quickstep onboarding tip window type to"" into pi-dev

StatsPullerManager not use singleton

This is to be consistent with other patterns such as UidMap.
This also makes unit test simpler.

Change-Id: I1558cd609e470481f269ecf2ae616277a95cfbf0
Bug: 72722120
Test: unit test

Merge "Have proper permission check in getConfigForSubId"

Make testSplitscreenPortraitAppOrientationRequests pass to unblock release am: cf2d6475f9
am: 6b99b6e3a3

Change-Id: Ia5697a4cae08cf56f94244db1445b92929e4dde9

Fix ManifestMerger issues

Set LOCAL_MIN_SDK_VERSION in SettingsLib so that the build system
doesn't raise the minSdkVersion value specified in the manifest,
which causes ManifestMerger to error when trying to merge it into
a package that specifies a lower minSdkVersion.

Bug: 78447299
Test: m java
Change-Id: Ie2c2458647cd6495418999b35894dc92343992cc

Make testSplitscreenPortraitAppOrientationRequests pass to unblock release
am: cf2d6475f9

Change-Id: I93ac74e692e5d0f7f78d2c431b9e4fd4300aec09

[automerger] Make testSplitscreenPortraitAppOrientationRequests pass to unblock release am: cf2d6475f9 am: 2dfb03f6f4

Change-Id: Idc3ae3c6d91f93a48a134280e176452ce050f21e

Make testSplitscreenPortraitAppOrientationRequests pass to unblock release
am: cf2d6475f9

Change-Id: Ie0862784fd537c0a669d3085b2d86a7ab482db7f

Merge "Import translations. DO NOT MERGE"

Merge "Clear calling identity before doing any work" am: 31686e954a am: 3eb2ad1096
am: ed66a8b2a6

Change-Id: I412d623de4ae2c32e0070bcb105010c234fe56b7

Merge "Clear calling identity before doing any work" am: 31686e954a
am: 3eb2ad1096

Change-Id: I3a940e1fb84bc04b7b53cbe25111768deb3e6d0c

Merge "Clear calling identity before doing any work"
am: 31686e954a

Change-Id: I144fd2b5f46ce7d7a06195a2dd5207d44cd951f0

Merge "Uses autofill callback to throttle performance tests."

Merge "Clear calling identity before doing any work"

Abbreviate permission labels less aggressively

We switched the package installer app to always abbreviate all labels
loaded from foreign packages. This is done so that a foreign app cannot
crorrupt the UI via super-long strings.

The default abbreviation is very aggressive and abbreviates to ~5 cm.

When an app with a targetSDK <= M gets installed, all permissions will
get auto-granted. Hence when a user side-loads such an app, we show the
permissions during install so that the user is aware of this.

Unfortunately most permission labels and restrictions do not fit into a
single line.

This change allows much longer strings for permissions names and
descriptions. It still abbreviates eventually and handles permission
descriptions with weird characters (such as back-space) correctly.

Fixes: 110209410
Test: Side-loaded old app and verified that permission labels and
      descriptions are not abbreviated.

Change-Id: Ie1db240883c9f5cb9c1b74ca7bdcb80625e3f7f2