Merge "Cleans up sharedUser regardless of creation logic" into qt-dev
am: a1bf282e43

Change-Id: Ic721f31332bbcbb9d64e501b3b94b6b457c8af51

Merge "Cleans up sharedUser regardless of creation logic" into qt-dev

Cleans up sharedUser regardless of creation logic

Prior to this change, we were not updating the shared user a package
setting belongs to if the package setting was not deep copied. In the
case that shared users didn't match, we were not deep copying and so
were not updating later. There is a more fundamental change to be made
here, but we're limiting risk by keeping this change as small as
possible.

Bug: 138919405
Test: atest AppSecurityTests
Test: manual; flash P then flash this build: observe consistent PackageSettings
Change-Id: Ib22f9e6a1fb3104bec4eb3e5b1de983e19d49dce

Merge "docs: Removed dates in obsolete field messages." into qt-dev
am: 45855214b5

Change-Id: I549026ea364645a8d60a1a06df9b20886a6df97a

Merge "docs: Removed dates in obsolete field messages." into qt-dev

Merge "Stop idmap2d after several seconds pass" into qt-r1-dev

Merge "GestureNav: Fix AbsSeekBar thumb exclusion" into qt-r1-dev

GestureNav: Fix AbsSeekBar thumb exclusion

Fix two issues that caused the exclusion for SeekBar thumbs to be to small and offset
from the thumb:

Account for padding and thumb offset; the thumb drawable is drawn with an offset
from the View's Canvas; the same offset must be applied when udpating the exclusion
rects.

The thumb is typically much smaller than the drag zone; the thumb rect alone doesn't
provide an appropriately large exclusion for reliably hitting it, so it is enlarged
to the height of the seek bar (up to 48dp).

Bug: 138992366
Test: manual, show exclusion zones with: adb shell setprop debug.pointerlocation.showexclusion 150 && adb shell settings put system pointer_location 1
Test: atest android.widget.AbsSeekBarTest
Change-Id: I2b670c6f3f33451bdccdfd3d75a75e90260257ff

[automerger skipped] Merge "Import translations. DO NOT MERGE" into qt-dev
am: 6aa3f38983 -s ours
am skip reason: subject contains skip directive

Change-Id: I9bb07364b95726b5e1b1f447f83ec5f9443706d6

Merge "Import translations. DO NOT MERGE" into qt-dev

Merge "Import translations. DO NOT MERGE" into qt-r1-dev

docs: Removed dates in obsolete field messages.

Notices about obsolete fields and methods now mention the publish
location (Google Play Store) or Android version (Q and higher).

Test: make ds-docs

Bug: 122482744
Change-Id: Idb60fef711401d43af8eb71343f157590c311571

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I177c5cfe63c60448882e1849f76a96d9103ed305

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ic7853a4d8574d496f1ee03a8163fff2671cafafd

Merge "Stop timer when notification is not visible" into qt-r1-dev

Stop timer when notification is not visible

This combines ag/8888937 and ag/9019277 into one CL, plus fixes another issue
where the timer could restart if the user had removed a notification and then
restarted the stream

Fixes: 138261464
Test: manual, atest NotificationMediaTemplateViewWrapperTest

Also checked the changes on the master branch since that had very
obvious issues when locking / unlocking the phone before

Change-Id: I6a0bbc675d33a5c7d4ce7f0884aec19606bff4fa

Merge "docs: Clarified when onRestoreInstanceState() executes" into qt-dev
am: c80fe63001

Change-Id: I332332fc88e565a3842297b04769778ee1c344ca

Merge "docs: Clarified when onRestoreInstanceState() executes" into qt-dev

Merge "[RESTRICT AUTOMERGE] CameraServiceProxy: More granularity to NFC / Camera interaction" into qt-r1-dev

[RESTRICT AUTOMERGE] CameraServiceProxy: More granularity to NFC / Camera interaction

Instead of disabling NFC active polling on all camera use, or never
disabling NFC active polling, support disabling based on camera facing.

Test: 'adb shell dumpsys nfc' while trying all 4 possible NFC options and
   using front/back cameras via default camera and via face authentication.
Bug: 138851393

Change-Id: I81eb014f0c2058258384538dce7c01b575c44df4

Merge "Only show the thermal shutdown warning when thermal reboot" into qt-r1-dev

Merge "Import translations. DO NOT MERGE" into qt-r1-dev

Only show the thermal shutdown warning when thermal reboot

showThermalShutdownWarning() is on the path where systemui starts
and restarts. Previously, we just checked the boot reason in
determining whether to show the warning, which makes it unexpectedly
seen when systemui restarts from crashes.

This CL caches the last boot count and compares with the current one
to ensure that there is a thermal reboot before the warning is shown.

Test: atest PowerUITest
Bug: 138089094
Change-Id: I0a51d00703d291ba10a6ab4f348308a14000a20e

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ief52f624ef932c1941ae4bb5a8264e57bd0495f5

Merge "Add a documentation in EmergencyNumber#getNumber" into qt-r1-dev

GestureNav: Log exclusion rect heights

Logs the rejected and requested exclusion rect heights.

Test: adb shell device_config put android:window_manager key_system_gesture_exclusion_log_debounce_millis 1000
Bug: 135152789
Change-Id: Ie12640f4bf0a8b40b666e604188a0bedd85e8981
Exempt-From-Owner-Approval: DeviceConfig.java: Adding constant to WindowManager namespace.

Merge "Fix overlapping remote and recents animations" into qt-r1-dev

Fix overlapping remote and recents animations

Test: Open app and immediately swipe up while opening. Verify
Thread prioties in systrace

Test: WindowProcessControllerTests
Fixes: 137881181
Change-Id: Ic1d7403d2a648e6f1d1b6a3d55b56bf37bc57a2c

[automerger skipped] Merge "Import translations. DO NOT MERGE" into qt-dev
am: ab6ef4f0df -s ours
am skip reason: subject contains skip directive

Change-Id: I6ddd3a2f1d738ea3b49e746ff102fe7515c9d50a

Merge "Import translations. DO NOT MERGE" into qt-dev

Merge "Import translations. DO NOT MERGE" into qt-r1-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: If0461bd95a9668d8582a37a65d9c37f3450723be

Merge "Acquire display suspend blocker for DozeService" into qt-r1-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I13841d4c409aed2b5192f6d5bf132a71f76da519

Merge "create ExclusionRectStateChanged/BackGesture atom" into qt-r1-dev

Merge "DO NOT MERGE Fix PermissionManagerService throwing exception for unknown package." into qt-r1-dev

Merge "Responsiveness improvement for BrightnessObserver" into qt-r1-dev

Merge "Add documentation in TelephonyManager#getNetworkCountryIso" into qt-r1-dev

Merge "Keep IME above starting window." into qt-r1-dev

Acquire display suspend blocker for DozeService

If DozeService explicitly requests the display state to be on,
then PowerManager needs to acquire the display suspend blocker even
while in DOZE mode to prevent the system from constantly trying
to suspend.

Bug: 138195405
Test: atest PowerManagerServiceTest
Change-Id: I05f5b86789ced084d0814480b2fe89d74f96a54e

Responsiveness improvement for BrightnessObserver

Forget the history to get immediate response when entering lower
brightness environment.

The sensor may not report event if there is no change since last report.
BrightnessObserver should keep refreshing the filter's estimation when last
sensor value is in a higher zone.

Bug: 138404221
Exempt-From-Owner-Approval: Santos is designated by Michael
Change-Id: I4ab37c067826c36eda26b435e83258d9af8783ab

Keep IME above starting window.

While app window is animating-in, IME window is child of aboveAppContainers.
If app requests IME on activity resume, IME will start animating-in
above Snapshot during app window animation.
However when app window animation finishes, IME's parent is no
longer aboveAppContainers, thereby making snapshot occlude
IME briefly. This results in flashing.

The fix is to keep IME in aboveAppContainers as long as startingWindow
is not removed from hierarchy.

Bug: 137534359
Test: Manual and atest ZOrderingTests AppWindowTokenTests
Change-Id: I78c939e8d1f23144162ba9eb8d834150fdd08ae8

Merge "Fixes breaking test" into qt-r1-dev

Merge "Fix "Invalid card" text on CarrierTextController" into qt-r1-dev

Merge "Don't tell Assistant to hide on gesture completion or cancel" into qt-r1-dev

Fixes breaking test

Fixes: 138812624
Test: atest SystemUiTests
Change-Id: If289ea1c2a0eaa2332d1328f3eb50acd71aaf354

Fix issue where SIM PIN message would be visible

We're resetting the view state when a config change happens, but
they can arrive even after the view is not visible anymore.

Let's not update the message area if we're GONE, since the message
area is shared with the Pattern and PIN pad views.

Test: manual
Fixes: 138751729
Change-Id: Idb4877f8ed22e8b640670bb9da66bb124a2f8166

docs: Clarified when onRestoreInstanceState() executes

This method executes only after onDestroy() is called.

Change-Id: I4880cc98ea08945f30b16baa7ae400a5bbe3e439
Test: make ds-docs
Bug: 136241777

Merge "GestureNav: Clarify javadoc in regards to exclusion limits" into qt-dev
am: c9f4bcae57

Change-Id: I0581305f01ebdaa121ce0e70ab9d44f142c9f99f

Merge "GestureNav: Clarify javadoc in regards to exclusion limits" into qt-dev

create ExclusionRectStateChanged/BackGesture atom

Bug: 135152789
Test: manual
Test: m statsd_testdrive -j31
Test: out/host/linux-x86/bin/statsd_testdrive [223|224]
Change-Id: Id4b36c89ed5bf3330795107fb6cd6750edcd0dbf
Merged-In: Id4b36c89ed5bf3330795107fb6cd6750edcd0dbf

Merge "Fix the race between entitlement start/stop" into qt-r1-dev

Add documentation in TelephonyManager#getNetworkCountryIso

In multi-sim, returns a shared emergency network country iso
 from other subscription if the subscription used to create the
 TelephonyManager doesn't camp on a network due to some reason
 (e.g. pin/puk locked), or sim is absent in the corresponding slot.

Test: unit test
Bug: 137236159
Bug: 137235431
Change-Id: I311e690d1f79ebefbc068c486d9bed814c129aab

Merge "Debug when pulses are dropped" into qt-r1-dev

Merge "Add retry logic for UNABLE_TO_PROCESS" into qt-r1-dev

Debug when pulses are dropped

Test: adb shell dumpsys activity service com.android.systemui
Test: adb logcat | grep NotificationInterruptionStateProvider
Bug: 138336424
Change-Id: I57c0d059e139991a67bdb85dba6f1e81382806dc

Merge changes from topic "dynamic_privacy_fixes" into qt-r1-dev

* changes:
  Avoid strange animation when dynamic privacy changes
  Improved the experience with bypass and dynamic privacy

Avoid strange animation when dynamic privacy changes

The dynamic privacy state would briefly switch during
unlocking, leading to the view appearing even though
it was already visible.

Fixes:  138620551
Test: enable dynamic privacy, unlock with face, then unlock by dragging down, observe no weird animation
Change-Id: I3cc0945af0b83ee18cd173545a099af45b442e56

Improved the experience with bypass and dynamic privacy

The user can now use the expand buttons and also clear
all is invisible while locked down.
The can also drag down normally to reveal the
contents of a notification.
Also, the section clear button is also properly updated
once dynamic privacy changes.

Fixes: 138671060
Fixes: 138669307
Test: drag down with notif hidden and bypass on on a sensitive notification
Change-Id: I8743400d0b46583802a8663f6723dfa6beee95c9

Merge "DO NOT MERGE Fix migration for default browser." into qt-r1-dev

Merge "Use alpha instead of show/hide when creating animation leashes." into qt-r1-dev

Merge "Sync up Home Handle and Assistant Handle fade" into qt-r1-dev

Merge "Add carrier config for data roaming notification" into qt-r1-dev

Merge "Remove icon name from NR dump" into qt-r1-dev

Sync up Home Handle and Assistant Handle fade

Test: Tested locally
BUG:135948401
FIX:135948401
Change-Id: I38917bf16000e723987ca835aa1784b16787a0b4

Merge changes from topic "remap-code" into qt-r1-dev

* changes:
  Add disconnect causes for WFC use cases.
  Add MEDIA_TIMEOUT error for RTCP timeout

Merge "Mitigate IPC calls from MultiUserSwitcher" into qt-r1-dev

Fix the race between entitlement start/stop

For very quickly hotspot OFF/ON, there is race between tethering
start entitlement from Settings and stop entitlement from wifi's
intent broadcast. Entitlement would only be started when Settings
call #startTethering(). But there are two points that tethering
would stop entitlement. One is #stopTethering() from Settings.
Another one is wifi tell tethering that it is shuting down.

In this change, second point of stop entitlement is removed to avoid
this race problem. Entitlement may have chance that do not be stopped
(e.g. network reset). This is not great but the behavior is the same
as P.
TODO: Don't reply on caller to stop entitlement. Refactor the design
to call #stopProvisioningIfNeeded when IpServer leaving TETHERED state.
TODO: mWifiTetherRequested may also suffer this race condition.

Bug: 137034691
Test: -build, flash, boot
      -atest FrameworkNetTests
      -Enable hotspot with entitlement supported SIM,
       then changing the AP band.

Change-Id: I63f858356b4543b5bdda41d03e45adc944b86da3
Merged-In: I63f858356b4543b5bdda41d03e45adc944b86da3

Merge "Update WebSettings.setSaveFormData deprecated doc" into qt-dev
am: 7bb64fcf27

Change-Id: Ife82891152d177e108db79f42fbe768fe8fb905a

Merge "Update WebSettings.setSaveFormData deprecated doc" into qt-dev

Merge "docs: Added missing comma in RectF class description." into qt-dev
am: 0c77665c37

Change-Id: I1450636a7ee9d59b75d95ec4e973813319f92d44

Merge "Add carrier config for hiding TTY HCO and VCO" into qt-r1-dev

Merge "docs: Added missing comma in RectF class description." into qt-dev

Merge "Fixed issue where camera wouldn't launch" into qt-r1-dev

Merge "Fix application overlay visibility bug" into qt-r1-dev

Add carrier config for data roaming notification

This config, when enabled, shows the user a notification when the device
switches to data roaming and the user has data roaming on.

Bug: 133358875
Test: manual
Change-Id: Ia091575ff0fa7e02f0c6a98bae0e66ef73beff81

Merge "Use shell package name when starting activities" into qt-r1-dev

Fixed issue where camera wouldn't launch

keyguardIsShowing, passed to NPV#canCameraGestureBeLaunched had the
wrong value because it was checking if SB was expanded visible
but ignoring the fact that bouncer would make it collapse.

Test: manual
Fixes: 138269798
Change-Id: Id259d42cbd7802614318d8f69c80470419f3b54b
Merged-In: Id259d42cbd7802614318d8f69c80470419f3b54b

Merge "Import translations. DO NOT MERGE" into qt-r1-dev

Fix application overlay visibility bug

System UI components may set the
SYSTEM_FLAG_HIDE_NON_SYSTEM_OVERLAY_WINDOWS flag in
WindowManager.LayoutParams to force application overlays to be
invisible. StatusBarViewController is responsible for setting this when
the user locks their phone, and then removing it when the user unlocks
their phone.

Currently there is a bug where, while StatusBar removes this flag, the
update is not propagated through WindowManagerService. This causes
application overlays to be invisible when they should not be. This
affects all application overlays.

Repro steps:

1) Enable rotation if disabled
2) Enable an application overlay (e.g. Live Caption)
3) In portrait orientation, open e.g. YouTube.
4) Rotate to landscape orientation
5) Turn off the phone screen
6) Rotate to portrait orientation
7) Turn on the phone screen and unlock the phone

Application overlays are now invisible. Rotating again fixes it.

This CL fixes the logic error that was causing this bug.

Bug: 133747592
Test: go/wm-smoke
Change-Id: Idab0629765cbe048fc1eb4413e529464cad876ff

DO NOT MERGE Fix PermissionManagerService throwing exception for unknown package.

The package might have been uninstalled asynchronously because we are
not holding the mPackages lock, so just log the error and return.

Fixes: 138670149
Test: presubmit
Change-Id: I7feeea85d2ab446e83e2101833e448cc962dd448

Merge "Treat SysUI as top application when GlobalActions is shown" into qt-r1-dev

Merge "Revert "Stop timer when notification isn't visible"" into qt-dev
am: 628f975c22

Change-Id: If351a368f5a1132e61dab1e7a1b1d5e290b092f7

DO NOT MERGE Fix migration for default browser.

Fixes: 124382812
Test: manual
Change-Id: I211e0dbba40b393d0827467ba4a9c979eb46131f

Merge "Revert "Stop timer when notification isn't visible"" into qt-dev

Revert "Stop timer when notification isn't visible"

This reverts commit 3c4a8e48d404649bd03ee8e6e377dcf9a65afee9.

Reason for revert: NPE errors
Fixes: b/138697134

Change-Id: Ic89eb58cd68485dcc5e7c5a3ef48d7ed79c303ca

Merge "Do not set AccessPoint.ssid to config.SSID for Passpoint networks" into qt-r1-dev

Merge "Do not gate bypass on isHardwareAvailable" into qt-r1-dev

Merge "Separate out sysui state updating" into qt-r1-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ifa499e736c7a4e42fe9ddfb6f32a5f7eefd8b6ac

Merge "syncInputWindows for mouse events" into qt-dev
am: 2774eb6635

Change-Id: I8c392689f632c50c3e18cbc1a1846c3c028c5f91

Treat SysUI as top application when GlobalActions is shown

This is done in order to reduce jank when showing GlobalActions.

Fixes: 136794922
Test: manual - launch GlobalActions, verify reduced jank
Change-Id: I2fe26dbb2aeb6d0fffd3b488a73370458b111cac

Fix "Invalid card" text on CarrierTextController

If there's an invalid sim that does not have a corresponding sub, the
text "Invalid card" should be appended to the current carrierText. As
this was assembled after this step (when it was empty), it ended up
replacing the correct carrierText instead of being appended.

Test: atest CarrierTextControllerTest
Bug: 138605164
Change-Id: Ibe0e2c3a218ea5c50f94e79cc46bd985b9646af0

Merge "syncInputWindows for mouse events" into qt-dev

Merge "Switch preffered display mode on lock screen" into qt-r1-dev

Mitigate IPC calls from MultiUserSwitcher

Reduce the calls to UserManager#hasUserRestriction when checking if the
switcher should be shown by reordering the conditions. In most cases,
the switcher is disabled (from settings) and it will short circuit.

TODO: actually remove the call from main thread

Test: manual, when device does not have multiple users, there are no
calls to UserManager
Bug: 138640355
Bug: 138661450

Change-Id: I35905d4d8a7194866f55009b86dff0a2a4214e1a

Add disconnect causes for WFC use cases.

Add new disconnect cause used when an attempt is made to place an
emergency call over WFC but the service is unavailable.
Add new disconnect cause when WFC is not available in an area.

Bug: 128993720
Test: Unit tests added to confirm remapping.
Change-Id: I15dcff8fabc06b5d469d083b0970fe0e77e7d318

Merge "[DO NOT MERGE] AudioService: fix disconnect/connect of A2DP device" into qt-r1-dev

[DO NOT MERGE] AudioService: fix disconnect/connect of A2DP device

 Due to the expected BECOMING_NOISY behavior associated
with a device disconnection, the disconnection is handled
asynchronously after a fixed delay. This delay caused an
inversion of commands in the processing order of the
disconnection of a device closely followed by connection
of the same device.
 The fix consists in:
- overriding the equals() operator for BtDeviceConnectionInfo
 so messages for a given device in the message queue
 can be checked / removed.
- when AudioDeviceBroker receives a command for A2DP
 connection or disconnection, remove all upcoming connection
 and disconnection commands in the queue for this device
 (see postBluetoothA2dpDeviceConnectionStateSuppressNoisyIntent)
- remove AudioDeviceBroker.handleSetA2dpSinkConnectionState, which
 was only used in BtHelper.onA2dpProfileConnected() with
 a CONNECTED state, and have this method perform a regular device
 connection (just like when coming from AM->AS).
- in AudioDeviceInventory.onSetA2dpSinkConnectionState(), support
 receiving a connection event for an already connected device,
 to support codec changes.

This change also includes modifications to the classes involved
in the device connection to make them support mocking/spying
to reproduce the bug conditions (see AudioDeviceBrokerTest).

Bug: 134932649
Test: atest AudioDeviceBrokerTest
Change-Id: If2b3b41409c77467a181a2f9b42310db9b9de8c5

Do not gate bypass on isHardwareAvailable

This is wrong for a few reasons:
- IPC can fail
- FaceManager can return false even when the device supports face auth
- UX should not change based on hardware problems

Fixes: 138658108
Test: manual
Change-Id: Ic736650ebcac3feacc360672242a0648ebe551ed