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

Merge "Only supports ambient mode when necessary." into qt-dev

[DO NOT MERGE] Don't drop restricted permissions on upgrade

Restricted permissions cannot be held until whitelisted. In
a P -> Q upgrade we grandfather all restricted permissions.
However, the whitelisting code runs after the internal update
of permission happens for the first time resulting in a
revocation of the restricted permissions we were about to
grandfather.

The fix is to not deal with restricted permission when updating
the permissions state until the permission controller has run
the grandfathering logic and once the latter happens we do run
the permission update logic again to properly handle the
restricted permissions.

Bug: 138263882

Test: atest CtsPermissionTestCases
      atest CtsPermission2TestCases
      atest CtsAppSecurityHostTestCases:android.appsecurity.cts.PermissionsHostTest
      P -> Q upgrade preserves grandfathered restricted permissions
      P -> Bad Q build -> Q fixes up broken fixed restricted permissions

Change-Id: Iaef80426bf50181df93d1380af1d0855340def8e

Merge "Do not allow the FalsingManager sesion to start when AOD is showing." into qt-dev

Do not allow the FalsingManager sesion to start when AOD is showing.

Bug: 138220274
Test: manual

Change-Id: I6d14db66d53a668b7cf8dd15ebb489fb3ebf4c7c
Merged-In: I6d14db66d53a668b7cf8dd15ebb489fb3ebf4c7c

Merge "docs: Fixed typo in SERVICE_META_DATA" into qt-dev

Merge "Revert "SharedMemory: break Cleaner reference cycle." Bug: 138422309" into qt-dev

Merge "Revert "SharedMemory: clear file descriptor when explicitly closed." Bug: 138422309" into qt-dev

Revert "SharedMemory: break Cleaner reference cycle."
Bug: 138422309

This reverts commit 390d9e6a1806626eb521d55a36b1578d28714cc8.

Reason for revert: crashes documented in b/138422309

Change-Id: I235f727d0fe87c09f6f05dddcae7759bab64dfd8

Revert "SharedMemory: clear file descriptor when explicitly closed."
Bug: 138422309

This reverts commit 20ab1e34273aa179053f5dc93e70c0191a39e91b.

Reason for revert: crashes documented in b/138422309

Change-Id: Ic9e33fdb24bad2b30f0eb357d6752c1834df41d5

Stop timer when notification isn't visible

Fixes: 138261464
Test: manual, atest com.android.systemui.statusbar.notification.row.wrapper.NotificationMediaTemplateViewWrapperTest
Change-Id: Id7b1b586f3ab258fad8670070240ef951080c969

Only supports ambient mode when necessary.

We only need support ambient mode on specified devices but we currently
enable this attribute on all devices, that means we need redraw every
time when ambient mode changes. However, we can just show / hide
wallpaper window on the devices which not support ambient transition.

In addition, always use portrait dimension as scissor area since we only
do transition in portrait.

Bug: 138021396
Bug: 137962047
Test: Manually test on F2, C1, B4, S4 and walleye.
Test: Set an image wallpaper, switch between home, aod and lockscreen.
Test: Also use finger print to unlock from aod to home directly.
Test: Rotate to landscape, set a new image wallpaper.
Test: Switch between home, aod and lockscreen.
Change-Id: I36dd49363e81df5a260b10695d90aa9d8c70a45a

Merge changes I8bdb4c74,I613a7d03 into qt-dev

* changes:
  SharedMemory: clear file descriptor when explicitly closed.
  SharedMemory: break Cleaner reference cycle.

Merge changes from topic "chromehole" into qt-dev

* changes:
  Only consider gone for layout if parent is gone for layout
  Reland "Prevent dismissing starting window when reopening app"

SharedMemory: clear file descriptor when explicitly closed.

We run the Cleaner in close, but after the fix in commit 6ca916a6, this
no longer clears the value stored in the FileDescriptor, which means
that subsequent operations on an explicitly closed SharedMemory will
operate on a bogus fd number. Clearing the FileDescriptor value in close
is sufficient, because Cleaner.clean is idempotent, and the only other
case where it executes is when the FileDescriptor is phantom reachable,
which means no one can access it to get its integer value.

Bug: http://b/138392115
Bug: http://b/138323667
Test: treehugger
Change-Id: I8bdb4c745466532a0712976416184c53fcf0dbf6
(cherry picked from commit a7641806ddf1099239632d53c629c062ff2168f4)

SharedMemory: break Cleaner reference cycle.

Previously, the Cleaner we create to close the ashmem file descriptor
used a thunk that held a strong reference to the FileDescriptor we
wanted to clean up, which prevented the Cleaner from ever running.

Break the cycle by storing the integer value of the file descriptor
instead.

Bug: http://b/138323667
Test: treehugger
Change-Id: I613a7d035892032f9567d59acb04672957c96011
(cherry picked from commit 6ca916a657cd56158212a57601108716ce78cbe8)

Only consider gone for layout if parent is gone for layout

If we check getParentWindowHidden, that determines mostly actual
visibility. However, we don't want that because we still would
like to follow the parent's window layout lifecycle, as otherwise
we may be stuck in a transition in case the parent window is
hidden but the child is waiting for a layout to happen.

Test: Click "Customize" on wallpaper picker, go back, ensure no
transition timeout
Fixes: 135976008

Change-Id: I66aeab29a81cd82b170aaf337249616b1f559848
(cherry picked from commit b52b0457e1bad14697341cb81f6d391755b009be)

Merge "Only update overlays for target" into qt-dev

Merge "Prevent the BrightLineFalsingManager from duping its session." into qt-dev

Merge "Fix issue with media notifs being misbucketed" into qt-dev

Merge "DO NOT MERGE Do not call drawableChanged pre Q" into qt-dev

Prevent the BrightLineFalsingManager from duping its session.

The BrightLineFalsingManager should not be able to start a session
when it's already in a session. Primarily, this caused the
FalsingManager to hang onto extra registrations to the Proximity
Sensor, per the bug.

Bug: 138220274
Test: atest SystemUITests and manual.
Change-Id: Id10d2697a96524e98c87aaa87702209d1752fe68

DO NOT MERGE Do not call drawableChanged pre Q

Some apps rely on not updating the window format when changing the
background of the DecorView. To keep the compatibilty with these app we
add only call DecoreView.drawableChanged() when the window background is
changed on app targetting Q and above.

Test: Manually test by lunching Instagram TV and pressing return twice.
The window should aninate with no flickering.
Bug: 136987724

Change-Id: I3593d30dc6f10519008151974e475f0dad86fc64

Fix issue with media notifs being misbucketed

Previous code assumed that "isHighPriority" == "is in top section",
which is not always true. Media notifs and some other notifs can appear
in the top section even if they're not high priority. Because we detect
section boundaries by iterating through the list until we find the first
notif where isHighPriority == false, we were sometimes drawing the
section boundary way too high. This change creates a new propery,
isInTopSection() that accurately tracks this state.

Setting this value in the proper location would require some seriously
destabilizing refactors, so instead we set it in the list comparator,
which is awful but here are.

Test: manual
Bug: 138320173
Change-Id: I19223720bac534ab92219a2962169097819e8efb
Merged-In: I19223720bac534ab92219a2962169097819e8efb
(cherry picked from commit 8c1b763dcf5df6e7fc7177dad5e7390adb229fa4)

Only update overlays for target

When a target package would update, the OMS would update the stat of
overlays that target android as if the android overlays targeted the
target package. This would cause the overlay state to change to
STATE_MISSING_TARGET for android overlays.

This change will only update state for overlays that target the updated
package.

Bug: 137038000
Bug: 136549878
Test: manual debugging
Change-Id: I0430426d8e8588a0fb97096a1f2570a7930069b3
Merged-In: I0d545253179b3545307b88b3151d11fd9e6cbd5b

Merge "Allow BrightLineFalsingManager settings to be experimented with via Phenotype." into qt-dev

Allow BrightLineFalsingManager settings to be experimented with via Phenotype.

This is a CP of http://ag/8687829

Bug: 138308096
Test: atest SystemUITests
Change-Id: I9e2b22b157c45da1606466acdfff3c5de7f182e1
Merged-In: I9fa4d1344bb184dea00f92f8d265667f0be11466

Merge "Don't animate clock when turning screen off." into qt-dev

Merge "Mark screen rotation as early (1/2)" into qt-dev

Mark screen rotation as early (1/2)

To decrease jank while rotating.

Test: Rotate screen
Bug: 138083790
Change-Id: If0b4428253dbfa4dcb102b8146a847e2e9ab876b
Merged-In: If0b4428253dbfa4dcb102b8146a847e2e9ab876b

Don't animate clock when turning screen off.

This change stops the animation because there isn't a
transition from the no-header state to the music header
state when turning the screen off (to AOD). Since there
isn't a transition, there isn't an animation.

This assumes that there isn't a transition from unlocked to
lock screen. If there is, then there would be an animation
of the music going away while arriving at the lock screen.

Fixes: 137383007
Test: Checked repro steps in bug, clock doesn't animate.
Test: Also checked repro steps when audio is paused, clock doesn't
animate.
Test: atest KeyguardSliceProviderTest.java

Change-Id: If39777340b72bc623d6690bc4f784c7f5c26ea8d
Merged-In: If39777340b72bc623d6690bc4f784c7f5c26ea8d

Merge "Fix permission javadoc about whitelisting." into qt-dev

Merge "Add cleanup method to FalsingManager to prevent memory leaks." into qt-dev

Merge "Fixed a bug with the animation of the notification background" into qt-dev

Merge "docs: Added protection level to permissions" into qt-dev

Merge "docs: Fixed link in setCurrentTab() description" into qt-dev

Fix permission javadoc about whitelisting.

Fixes: 138122977
Test: presubmit
Change-Id: I8dec56e4385877789ea3504bc1efbe59ad0ddbb7

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

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I8688211f4bb949f51359257a442b81d62e7057bf

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I9e12ebb9c597b7393a32d78a7b15c830326c2915

Merge "docs: Fix broken link." into qt-dev

Add cleanup method to FalsingManager to prevent memory leaks.

When the FalsingManager gets reloaded (due to plugins) it can
leak its listeners and callbacks. This change fixes that.

This is a CP of http://ag/8668802.

Bug: 136351609
Test: manual
Change-Id: I7ea8384678b60e78ed384e19bbd7932722fe2b9c
Merged-In: I2b52d018d478dbcad4ecb7d8a5b361638d5c5877

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

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I9041213adcd88452a12b56e110017a314f7a8139

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I5bc0c666b5733a049ecffef86891a1d564bfdfc8

Fixed a bug with the animation of the notification background

The position of the background could be 0 and therefore the
background could fly over the whole screen.
We're now positioning the top section at the start
of the first visible section.

Fixes: 137885070
Test: atest SystemUiTests
Change-Id: I6720b570f5cd209249b90681cdf6c6a56cf0194e

Merge "Fixed a crash with the headsUpManager" into qt-dev

Merge "Use binned brightness sensor instead of prox" into qt-dev

Merge "Enable the BrightLineFalsingManager by default on Q." into qt-dev

Use binned brightness sensor instead of prox

Prox can be noisy and should only be used as a fallback when a more
robust sensor implementation is not available.

Test: manually cover prox sensor
Test: breakpoint
Test: dumpsys
Fixes: 137451005
Change-Id: If0fca42aca36546942930ce76be1353129fc91fc

Fixed a crash with the headsUpManager

Since the headsUpManager is calling out in various places to
its listeners, the callbacks may query the headsupmanager
in an internally inconsistent state, such that the pinned mode is
true but there is no topEntry. Let's add a null-check for
safety here.

Bug: 137804505
Test: atest SystemUITests
Change-Id: Ibae76b555ca51ccf676228b034a614d59a8b4e8e

Merge "Postpone the shared user checking after Q" into qt-dev

Merge "docs: Fix broken link." into qt-dev

Enable the BrightLineFalsingManager by default on Q.

Bug: 71762354
Change-Id: Icfc6e21d50e08c34168f504ec64bf654ba6cb6a4
Test: atest SystemUITests

Postpone the shared user checking after Q

The shared user checking becomes strict. If there is an APK in
vendor partition shared UID with a APK in system partition, the
signatures of these 2 APKs must be equal. Otherwise, framework
will throw an exception and cannot boot.

Postpone the checking from first_api_level 28 to 29.

Bug: 137524152
Test: test on a device with first_api_level=29 and pass the checking
Change-Id: I76d87a6dab35d4d39113cdd16a20706ce2c45093

docs: Fix broken link.

Test: make ds-docs

Bug: 132635673
Change-Id: I9e6a493112418f7b1467d9d2926a1b2c106d4ad0

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

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

docs: Added protection level to permissions

All permissions that are suitable for use by third-party apps and
aren't deprecated now contain a paragraph in their descriptions that
lists the protection level for a given permission, such as:

Protection level: signature|privileged

Test: make ds-docs
Bug: 137260540
Change-Id: I45eada81bf4aa83480ff2e701a9e0e9932b66b88

docs: Fix broken link.

Test: make ds-docs

Bug: 79729313
Change-Id: I3f7b90457e9127380f2b8cbaf0f958823ba5488e

docs: Fixed typo in SERVICE_META_DATA

Test: make ds-docs

Bug: 137267707
Change-Id: I3f38362f028866f55b9009b6a0de15dcbcb96e0c

docs: Fixed link in setCurrentTab() description

Moving focus and selected tab requires focusCurrentTab(), not
setCurrentTab().

Test: make ds-docs

Bug: 137317608
Change-Id: I96f4f4c32cbf8a3dfa0aac8cfa6d6cd12e8f8259

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ib95137b7ade70cbb7184e3d0a7f004fc759bc064

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I10856a2e242d1c493f514e67884d26e4fd057447

Merge "p2p: update documents about EXTRA_WIFI_P2P_GROUP in connection changed event" into qt-dev

Merge "Do not animate wallpaper when wakeAndUnlock" into qt-dev

Merge "PasswordSlotManager: Fix incorrect string comparison." into qt-dev

p2p: update documents about EXTRA_WIFI_P2P_GROUP in connection changed event

This value may be a null, receiver should check before using variable.

Bug: 137618950
Test: make docs
Change-Id: If466dac092fc345482d0456f08a915d1e7ba0010

Do not animate wallpaper when wakeAndUnlock

Fixes: 137536016
Test: press power to go to lock scree <-> aod
Test: unlock with fingerprint
Test: trigger AOD2 pulse (look at wallpaper animation)
Test: atest DozeWallpaperStateTest
Change-Id: I46574609a00712bdde5a4652ab5c9b0c04cbeeec
Merged-In: I46574609a00712bdde5a4652ab5c9b0c04cbeeec

Merge "Fix CTS case failed in CtsAutoFillServiceTestCases" into qt-dev

Fix CTS case failed in CtsAutoFillServiceTestCases

The case failed due to AutofillManager can not find the ResolveInfo of AutofillCompatAccessibilityService.
AutofillCompatAccessibilityService is not visible for instant App.
Cts test process is working at other users. And the calling App is instant App.
So cts test process can’t get service info of AutofillCompatAccessibilityService.

Bug: 137236035
Test: run cts -m CtsAutoFillServiceTestCases
Change-Id: I4dffc615ea1d8290bc02e51a9aa055fa0ead467f

Compute policy for read-external-storage for uid

So that when sharding a uid all packages have the same policy.

Otherwise one app might want to set the app-op to a different state than
another app in the same uid which will lead to a loop where the two apps
fight.

Test: atest --test-mapping frameworks/base/services/core/java/com/android/server/policy
Fixes: 136129296
Change-Id: I673342f4513b35854faaab993fc4233fe19217d0

Merge "GestureNav: make reportSystemGestureExclusionChanged oneway" into qt-dev

GestureNav: make reportSystemGestureExclusionChanged oneway

When the WindowManager lock is contended, reportSystemGestureExclusionChanged can cause
jank. Making it oneway avoids this.

Fixes: 137650591
Test: make droid
Change-Id: I5d96b7e2225745b146014ad11cafaf0ad9a1b734
Merged-In: I5d96b7e2225745b146014ad11cafaf0ad9a1b734

Merge "AM: make isHighEndGfx TestApi" into qt-dev

PasswordSlotManager: Fix incorrect string comparison.

Bug: 135721162
Test: cts-on-gsi -m CtsWindowManagerDeviceTestCases
Change-Id: I2a11dd6dfd44e10400e40ebbaa2ed597de95cdcb
Merged-In: I2a11dd6dfd44e10400e40ebbaa2ed597de95cdcb

AM: make isHighEndGfx TestApi

Bug: 136784434
Test: n/a
Change-Id: Ie26de8dcf64a02b7757d4f705485cdcb7470f511

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

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

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Iead3ad19af56e9a6e2fef89a946b5f69c4b4674c

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

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

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ib19f01fa0ff2376e7db5168cc45d5da4321b28d8

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I382b136a55db6b38eb364f6f4934129b05411a53

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I424bc1a910c09f141985edcc4375202046ef50a4

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I166c16c2b9dc34c74780e41b1ac73612e79f46f7

Merge "Pass EXTRA_CALLING_PACKAGE for legacy change default dialer/SMS intents." into qt-dev

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

Don't persist new night mode setting in car mode

Keep persist.sys.theme in sync with the primary user's
setting value UI_NIGHT_MODE

Test: manual
Fixes: 137174308
Change-Id: I8df67e0734b08bd82418e381d153d9012ecf99e4
(cherry picked from commit 47a775b67de77317f6ad357a9c4a3d88da846253)

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I22c679764a454425947c3e0bd3949b18fa6678d4

Pass EXTRA_CALLING_PACKAGE for legacy change default dialer/SMS intents.

So that RequestRoleActivity can always know the calling package. This
is required to keep supporting pre-Q apps calling with
startActivity().

Also fixed application info retrieval to be aware of multi-user.

Bug: 137054465
Test: manual
Change-Id: I686a97b543a0218837ee08291ef6fd9579a551d1

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

Merge "Disable testEnableRollbackTimeoutFailsRollback test." into qt-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I9552d0718e0418dea093f7735268e9f917257e2f

Merge "Redact ISOBMFF boxes with 'free'" into qt-dev

Disable testEnableRollbackTimeoutFailsRollback test.

The test is flaky because it is testing a race condition.

The flakiness is fixed with the help of some new @TestApi in
I01f3e769711ffa51fa714ff4bc81e70887f427ad. Disable the test on those
branches without the fix merged.

Bug: 134373106
Bug: 136605788
Test: atest RollbackTest

Change-Id: I05edfc142c14631b259ae7aaf2dd31af00363a35
Merged-In: I01f3e769711ffa51fa714ff4bc81e70887f427ad

Merge "Use a Region when calculating TouchableRegion for cutouts" into qt-dev

Merge "Q card." into qt-dev

Merge changes from topic "am-491d8213-3eca-4563-937d-c6f23dd5f887" into oc-dev am: 4244190788 am: 61ea78238e am: 61581686b6
am: 8c5f02eddd

Change-Id: Ic2236ac466bd86e019b7c004e9498493e16b96b7

Merge "Fix NullPointerException." into qt-dev