[automerger skipped] DO NOT MERGE SurfaceControl: Fix captureLayers JNI
am: fbc22c2b6d -s ours
am skip reason: subject contains skip directive

Change-Id: Iedd5183586ec5b13c71ae943c3ff877e5c98311e

Merge "Allow non-convex paths in Outline#setConvexPath" into qt-dev

Merge changes I1360deb0,Id9a9eb85,I7f05f2ee,Ie48da88d into qt-dev

* changes:
  Fix issues with animations when the stack is expanded.
  Prevent a crash when touching the stack while it's flinging out of the dismiss area.
  Remove the max rendered children code.
  Stability and logic fixes to PhysicsAnimationLayout to address bugs that leave Bubbles in a bad state.

Merge "Add Usage info for LocationManager's APIs" into qt-dev

Merge "Adding config for SUW to determine whether to start Tips for gestural nav" into qt-dev

Merge "Skip IME initialization when received the token that already registered" into qt-dev

Merge "Exclude the status bar and HUN from gestures" into qt-dev

Merge "Handles pre-process failure without crashing system" into qt-dev

Merge "Constrain global actions panel to available space" into qt-dev

Merge "Make the Handle bar grabbable." into qt-dev

Merge "Add user-switching observer to AttentionDetector" into qt-dev

Fix issues with animations when the stack is expanded.

- Don't show the flyout if the stack has become expanded.
- Add onChildReordered so that the controllers don't receive onChildAdded calls when views are reordered. This was causing animation issues with expanded bubbles.
- Clear the dragging out bubble onGestureFinished, so that it's not ignored by animations.
- Update the expand/collapse animations when a new child is added during animation, so that it'll also animate to the proper spot.

Fixes: 129370170
Bug: 123542488
Test: atest SystemUITests
Change-Id: I1360deb09db82bd3ba72cb91fc9abe05b6dc1a9c

Constrain global actions panel to available space

Formerly, the global actions panel was given the entire screen, and
was rendered above the global actions menu. Now, we limit it to the
space unused by the menu.

Bug: 133511683
Test: manual
Change-Id: I74f4db0b9a2a4e030ec1fd53ddd65e7c372d229c

Merge "Add network health check watchdog triggered rollback tests" into qt-dev

Merge changes I2cb4200b,If1fa00be into qt-dev

* changes:
  PackageWatchdog listens for NetworkStack failures
  Proper handling of NetworkStack crash

Merge "Remove @VisibleForTesting from DevicePolicyEventLogger getters." into qt-dev

Handles pre-process failure without crashing system

This change ensures that if we encounter any issues copying the APK to
be installed, we also fail the other sessions to prevent them being
cleaned up incorrectly on post-install.

Test: manual; use debugger to force InstallArgs.copyApk() to fail
Bug: 133380744
Change-Id: I19e84a4683a36ac152d435fab3d0b191db6cc2df

Remove @VisibleForTesting from DevicePolicyEventLogger getters.

Also make it extendable for mocking and update atoms.proto comments.

Bug: 131833599
Test: compiled
Change-Id: I6a0e8a6175e7e9e46572ea4aa50ce7669a02588b

Allow non-convex paths in Outline#setConvexPath

Bug: 131759669
Test: I5eca77e1a60e484e4e118b7e464a88363c539ca9
Test: Manual - assert no longer fires and app looks normal

The limitation to only support convex paths was due to a limitation
in the old renderer. Today, it is fine to use a concave path. Further,
Skia has changed how it computes complexity - it is more conservative,
so paths which were previously thought to be convex are no longer. We
cannot guarantee that a path will be considered convex (especially
after e.g. rotating it, as in the library in question), so drop the
requirement.

Change-Id: Ice88d0995750e066320cb175a87f8ae70ce3aeed

Merge "Fix issue for DnsResolver#query" into qt-dev

PackageWatchdog listens for NetworkStack failures

In addition to the NetworkStack app monitoring, have PackageWatchdog
register an observer to NetworkStackClient to receive severe failure
notifications, and attempt a rollback if available.

The callback is registered in onPackagesReady(), which is called in the
boot sequence just before starting the NetworkStack.

Test: installed new networkstack, killed it twice, observe rollback
Test: unit test in change on top
Bug: 133725814
Change-Id: I2cb4200b78c2482cacc4bfe2ace1581b869be512

Proper handling of NetworkStack crash

Instead of always crashing on userdebug builds, do the following on all
builds:
 - If the device did not observe a NetworkStack crash in the last 6h
   crash the system server. This is to handle spurious crashes of the
   NetworkStack, so that the system can recover instead of staying
   without connectivity.
 - Otherwise, the device has had a recent crash. Notify the listeners
   (watchdog listener to be added) that something is seriously wrong,
   but do not crash to avoid bootlooping the device. This allows the
   watchdog to do its job, and avoids bricking the device in
   situations where the user could need to make emergency calls.

Bug: 133725814
Test: Killed the network stack, observe reboot of framework.
Test: Kill network stack again, observe nothing.
Test: Wipe data, install new network stack, kill it twice: rollback.
      (with patch on top applied)
Change-Id: If1fa00bed769eb60ca4832609006bdf15ceddb80

Merge "Replace limited connectivity notification after detecting it" into qt-dev

Merge "Fix some issues in notification bucketing" into qt-dev

Add network health check watchdog triggered rollback tests

This cl improves code coverage for the PackageWatchdog explicit health
checks.

Updating the NetworkStack triggers an explicit health check with
a time deadline. At the deadline, if network is unavailable on the device,
the update is rolled back, if network is available, nothing happens.

Bug: 132640467
Test: atest StagedRollbackTest

Change-Id: I16a40244de9345930a1c981188aa33546fe872fc

Merge "Always give VPN the INTERNET capability." into qt-dev

Fix issue for DnsResolver#query

1. Previously, getDnsNetId doesn't handle all the cases.
Fix it with cosidering bypass private DNS flag.
2. Make getDnsNetId return Network instead of netId,
and change name from getDnsNetId to getDnsNetwork

Bug: 129530368
Test: atest DnsResolverTest DnsUtilsTest

Merged-In: Ibb5080acd3c296650d56532fc7da525e9fa95e8f
(cherry picked from commit 3854966dc9499e39187835606397b16367e5e27b)

Change-Id: I37353642088bcc17da0cf17f78a5ed9efc9aefc3

Replace limited connectivity notification after detecting it

No internet notification may be prompted before partial
connectivity being detected. Partial connectivity status will
be set into NAI and prompted in the Setting. Behavior is not
aligned between Setting and notification. Thus, update
notification again if partial connectivity is detected.
Also, sliently show the updated notification if no internet
notification has already been shown to user to prevent alerting
user in short time.

Bug: 130683832
Test: Verified with simulated partial connectivity
Test: atest FrameworksNetTests
Change-Id: Ie16a8ce6e0fa437048e8c1eea240314ca30e9520
Merged-In: I004e78a33689e2208918d4316bcf9a8f50a0bac3
Merged-In: I14385a39d99a45c4a6a50a665f456f589c2f4da3
(cherry picked from commit a5c68348d89f256cb5f42283d983d05834c7e36c)

Merge "Check system_server PID in NetworkStack calls" into qt-dev

Merge "Catch OutOfMemoryError and log warning" into qt-dev

Merge "Fix captive portal can't show certificate problem" into qt-dev

Always give VPN the INTERNET capability.

Split-tunnel VPN (which are the only ones affected by this change)
always fall through to the default network for routes they don't
handle, and even if the underlying network(s) don't provide access
this may be a pinhole that can actually reach the broader network.

In practice this behaves like the original release of P and is the
safest thing to do for Q. In R we should evaluate giving the VPN
app the ability to simply tell the network stack whether it does
provide Internet access or not.

Bug: 119216095
Test: FrameworksNetTests NetworkStackTests
Change-Id: I262ca41fe0225660551c9a421562405366b6acac
Merged-In: I262ca41fe0225660551c9a421562405366b6acac
(cherry picked from commit 6d5a4a3b1d1c69eb8a542fecf5e8a306822c19b7)

Check system_server PID in NetworkStack calls

Add a check that callers with UID 1000 always have the same PID. This is
a proxy for checking that no system is designed to bind to the network
stack unless it is the system_server, as otherwise either the
system_server would start crashing, or that system would not have access
to binder calls.

Also remove access from PHONE_UID as it is not being used.

Test: Flashed, WiFi working, Bluetooth reverse tethering shows no
      permission issue.
Bug: 133209255
Change-Id: Ib848aaaedfd599c1d4437378846c7dda74352019

Merge "Adding a compat class to listen for exclusion rect changes" into qt-dev

Merge "Update sysui OWNERS" into qt-dev

Merge "Prevent installation of new APEX." into qt-dev

Fix captive portal can't show certificate problem

Captive portal overrides two shouldOverrideUrlLoading methods,
but show certificate logic only be implemented in deprecated one.

Bug: 133714666
Test: -build, flash, boot
      -atest FramworksNetTests
      -manual test

Change-Id: I2656bb709cb0bf259279ca75ad8411cd8c92fb43
Merged-In: I64b05adcb46baf3fca116dbdc5177bc31c71e06d
(cherry picked from commit 9161b5e845eb8aa9b5ecf0dd464bca73359e1553)

Catch OutOfMemoryError and log warning

Bug: 133827762
Test: atest CtsMediaTestCases:android.media.cts.ExifInterfaceTest
Change-Id: I5c6e247136e033968f418a50276971a8a686e021

Skip IME initialization when received the token that already registered

When IME window switches from current display to another display and back quickly,
system will re-bind InputMethodService to re-attach window token for each moving,
the IME service lifecycle will be onBind -> onUnBind -> onDestroy -> onBind.

Ideally, system will deliver window token when onServiceConnected and system should
only receive one onServiceConnected when the last service bound.

But due to user switch display very quick, before unbind service callbacked from client,
the next bind service request comes, caused the connection exists in system side and
client will then callback 2 onServiceConnected to system.

Since CL [1] introduced InputMethodPrivilegedOperationsRegistry to deal with token
registraction singleton-ness, the exception will be thrown for above case due to
the duplicate window token passed to client.

Add InputMethodPrivilegedOperationsRegistry#isRegistered to check if the token
already registered, if so, skip this duplicate initialization request.

[1]: I4a61470f06ffac5f7a512536f8431489db0108f4

Fix: 133624278
Test: manual as below steps:
    1) Settings > Developer options, enable "Simulated display" & "force desktop mode"
    2) Reboot device.
    3) Launch app from simulated display.
    4) Launch app from primary display (i.e. contacts), focus EditText to show IME.
    5) Tapping primary & simulated display repeatly & quickly.
    6) See if any exception log from IME application. (i.e. Gboard)

Change-Id: Ie6bfbae735724fe744590e715124d2737d2b665d

Merge "Update Build#SERIAL docs to reflect device ID restrictions" into qt-dev

Add Usage info for LocationManager's APIs

All new code in LMS wrapped with try/catch and swallowing exceptions
to make sure new code never crashes LocationManagerService.

Bug: 133906682,133190291
Test: Tested with (statsd_testdrive + gnss test apk). Work as intended.
Change-Id: I989b1361c029e76d35861cf23d5c3d152f0bf81a

Adds learned state to the Reminder Experience for Assistant Handles

Also sets the Reminder Experience as the default behavior.

Test: Tested locally
BUG:132983599
Change-Id: I27b063ca142fbde99b0bd1f6d107f45d6070869a

Merge "Add server information for pptp vpn" into qt-dev

Merge "Add white edgelights to AOSP" into qt-dev

Make the Handle bar grabbable.

Bug: 133514822
Test: Manual
Change-Id: I4bd9ab183ac5953d1e581031832512e7a9cc17aa

Merge "NetworkStatAccess Should checkCarrierPrivileges cross all subscriptions" into qt-dev

Merge "Hide and re-show assist handles when phone is rotated." into qt-dev

Merge "Adjustments to handles visuals and behavior" into qt-dev

Adding a compat class to listen for exclusion rect changes

Bug: 133844237
Test: Verified library changes with Launcher
Change-Id: I39e817943d3804353dbe17fd4b7339970181acb5

Update sysui OWNERS

- The original change already went into aosp-master, this is just to CP
  it back into qt-dev

Bug: 133878173
Test: None
Change-Id: I43640d87c4fcd0a8211dc04c4203a7b80b150f58
Merged-In: I43640d87c4fcd0a8211dc04c4203a7b80b150f58

Exclude the status bar and HUN from gestures

Bug: 133264268
Test: Ensure that HUNs and collapsed status bar prevents back
Change-Id: I5d326be8525ab959180c815f48c1275255a5a785

Merge "Fix MediaHTTPConnection.disconnect() blocking for a long time." into qt-dev

Merge "Log during one-time WebView migration." into qt-dev

Add white edgelights to AOSP

Moves most of the invocation code (as well as PerimeterPathGuide,
CornerPathRenderer, and EdgeLight) into base SystemUI. Shows
white edge lights coming in from the corners into the center, from
gesture or squeeze.

Bug: 132984557
Test: manual

Change-Id: Icbe611c3513f24f0ac13b68bd4d65f7cb4d402d6

Merge "Handle NATIVE_WINDOW_TRANSFORM_INVERSE_DISPLAY flag in ASurfaceTransaction" into qt-dev

Adding config for SUW to determine whether to start Tips for gestural nav

Bug: 133861720
Test: Just adding resources
Change-Id: Ibe68168c8e71b4d529353951470708e4f84c0ae8

Fix some issues in notification bucketing

1 - fix the isHighPriority method to not boost
notifications that users have manually demoted
2 - include the isHighPriority field in the notif sort
so that the divider can be drawn in the right place

Test: atest
Fixes: 133134820
Bug: 129772718
Change-Id: I2c8f6d81e17b50a5be3ffe498536f5f3e59a0dc9

Adjustments to handles visuals and behavior

- Fix display on devices with larger corners by resizing the handle
views as the corner radius changes.
- Fix home x assist handle color matching by ensuring `isListening` is
set to after color matching start
- Allow UiController to suppress handles showing up during assist
sessions by calling ScreenDecorations.setAssistHintBlocked
- Add animation curve from spec to handle scale animation
- Fix handle rounded ends
- Make handle dimensions customizable

jamesoleary 2 CLs - ag/7727795, ag/7727797 (fix test failure)
mrcasey 2 CLs - ag/7704742, ag/7713549

Test: Verify WAI
Bug: 130642504
Change-Id: I9563d7f437064c8c63c988a9fda41be906064219

Update Build#SERIAL docs to reflect device ID restrictions

Bug: 133724689
Test: Build
Change-Id: I1188a1b0201730a649e3fa479989ba9d1ec3a143

Merge "Close windows on ACTION_UP when touched outside" into qt-dev

Merge "Considered network preference when simulating legacy data RAT" into qt-dev

Merge "Applies INSTALL_REPLACE_EXISTING outside of while loop" into qt-dev

Merge "Add battery estimate to content description." into qt-dev

Merge "Fix getMediaUri() bug on secondary devices." into qt-dev

Merge "Update assist handle behavior control" into qt-dev

Prevent a crash when touching the stack while it's flinging out of the dismiss area.

Fixes: 133867082
Test: atest SystemUITests
Change-Id: Id9a9eb85eeb53002a23f62605b21beafe1c4338f

Remove the max rendered children code.

This code is redundant given the BubbleData's handling of the bubble limit, and is responsible for the invisible bubble bugs.

Test: atest SystemUITests
Fixes: 132690748
Change-Id: I7f05f2ee12183f812abb55bf133c11dafce10909

Merge "Update notification bell and bluetooth volume icon" into qt-dev

Stability and logic fixes to PhysicsAnimationLayout to address bugs that leave Bubbles in a bad state.

- Ignore attempts to start animations from controllers that aren't the active controller.
- Use -Float.MAX_VALUE for the start velocity default, not 0, so negative velocities aren't discarded.
- Update the SpringForce's final position directly rather than using animateToFinalPosition (the latter is susceptible to race conditions if animateToFinalPosition is called again before the next animation frame, a bug in DynamicAnimation).

Test: atest SystemUITests
Fixes: 131625234
Change-Id: Ie48da88db6c241896bba17bb800a2f91ab61651e

Merge "MediaFrameworkTest: Fix storage path" into qt-dev

Fix getMediaUri() bug on secondary devices.

We need to return visible paths for MediaProvider to work with,
instead of internal paths.

Bug: 133730441
Test: atest android.providerui.cts.MediaStoreUiTest
Change-Id: If06dc635b22032761626cf54b37b7bc15ace7b2e

Update assist handle behavior control

Improves behavior modes to use more reliable callback mechanisms. Added
phenotype flag hooks for control of behavior and behavior parameters.
Added more robust null guarding.

Test: Tested locally
BUG:132983599
Change-Id: I42039152836071941d6b4f1cafb3670c62a65824

Considered network preference when simulating legacy data RAT

Correctly supported the legacy data RAT when both cellular
and IWLAN are in service. In this case only report
RAT as IWLAN when any type of APN is preferred on IWLAN.
Otherwise still use the RAT reported by cellular network
service.

Bug: 133516945
Test: Manual

Change-Id: Ifdddad1d5ee95f4576d5002a1b66caafc172cb58

Add server information for pptp vpn

The pptp vpn set up in ip-up-vpn.c does not provide an environment
variable for the remote server so we cannot setup a correct throw route
to the VPN endpoint. Change the LegacyVpnRunner class to include the
remote server information in the class so it can use it to bring up the
vpn and set the throw rule correctly.

Bug: 133797637
Test: Build
Change-Id: Ia6b00dd88f8f31823647c56c20b8c7edd20280f7
Merged-In: Ia6b00dd88f8f31823647c56c20b8c7edd20280f7
(cherry picked from commit b38065dd3a22452572f4bd6c380ed2c7f73ac103)

Add battery estimate to content description.

Fixes: 128892060
Test: mp sysuig
Change-Id: Ic8563fc3681d695fdacc94ca03e96df898b40e0c

Merge "Filter disabled/suspended direct share targets in sharesheet" into qt-dev

Add user-switching observer to AttentionDetector

Bug: 133208762
Test: manually verified on device.
Change-Id: I98fa103ec131aa0c18c84921539867f11d8fc0e5

NetworkStatAccess Should checkCarrierPrivileges cross all subscriptions

Bug: 133236378
Test: Manual
Change-Id: I49fbde1fe73b33aadcf8fd23ad224f363b137bf3
Merged-in: I49fbde1fe73b33aadcf8fd23ad224f363b137bf3

Merge changes from topic "no_layers_profileable" into qt-dev

* changes:
  GraphicsEnvironment: No debug ANGLE packages for profileable apps
  Revert "GraphicsEnvironment: Allow profileable apps to load layers"

Merge "Add snooze options setting" into qt-dev

Merge "Add privapp config for cluster." into qt-dev

Merge "Changing default limits, clarify flags, fix bug." into qt-dev

Merge "Sharesheet - Fix NPE on default app launch" into qt-dev

Update notification bell and bluetooth volume icon

Fixes: 133836840
Fixes: 110213437
Test: mp sysuig
Change-Id: I8a102aa6a5c5347cae4dad0425b932be3dd7259b

MediaFrameworkTest: Fix storage path

Test: Run mediaframeworktest
Bug: 133767357
Change-Id: If2930dffff0582a6a1b33a7de15c3008aaf923bd

Merge "Fix NPE when querying for already uninstalled package." into qt-dev

Merge "Change suppresible type of voice comm signalling" into qt-dev

Merge changes from topic "vpn_data_accounting" into qt-dev

* changes:
  Add one more test for VPN usage stats.
  Addressing comments for http://ag/7700679.
  NetworkStatsService: Fix getDetailedUidStats to take VPNs into account.
  Take all VPN underlying networks into account when migrating traffic for VPN uid.

Applies INSTALL_REPLACE_EXISTING outside of while loop

Prior to this change, the default behavior of replacing packages unless
-R was provided would only happen if at least one command line argument
was provided due to the actual setting of the flag happening within the
while loop that parses arguments. This change moves the application
outside of that loop, ensuring it's always set by default, even if no
arguments are provided.

Bug: 133453175
Test: adb install-multi-package ... with existing packages
Change-Id: I6673bb92c318b0a328880a5e9eb7cc6f5ba0ff40

Hide and re-show assist handles when phone is rotated.

Not getting fancy with the animation for the moment, just making sure
they show and hide in the right places.

Test: Show handles, verify that they show when the phone is rotated.
Bug: 133836023
Change-Id: I7cda1e3f0cf9a46993bfd3e8da67311e55fb6189

Merge "BluetoothProfileConnector: Invoke service disconnections with the correct profile ID" into qt-dev

Merge "Changed comments to match real protection level" into qt-dev

Merge "Appropriately sets all installs on *any* failure of one" into qt-dev

Merge "Fix AppFuseMountScope.close()" into qt-dev

Merge "Add dark theme for Shell app." into qt-dev

Prevent installation of new APEX.

This was previously only done on user build and the code convergence
caused bugs that we haven't spotted for a while.

Change-Id: Ib7f51b5ef6d3f221483eb2e2dcb2f4d19a0fbbee
Fixes: 132429173
Test: APEX CTS tests on both user and userdebug builds

Fix AppFuseMountScope.close()

The boolean tracking whether the scope had been opened or not
was never set before. Set it in open(), as was the case in the
previous (non-binder) version of this class.

Bug: 132901939
Test: monitor `adb shell mount` while browsing photos and
      verify the appfuse mounts don't build up over time
Change-Id: Ia164ac4209126ed7e4d273c5fca4ababc0c66f2a

Change suppresible type of voice comm signalling

USAGE_VOICE_COMMUNICATION_SIGNALLING shouldn't be considered
SUPPRESSIBLE_SYSTEM. It includes the ring tone sound when in a call.
Instead, it should not be suppressible like USAGE_VOICE_COMMUNICATION.

Test: manual
Fixes: 131756661
Change-Id: Ide8aafafe58c421f3b699e6de9358b4e73ad01c5

Merge "Make statusBarColor and navBarColor transparent for Theme.DeviceDefault.Settings" into qt-dev

Sharesheet - Fix NPE on default app launch

Turns out there's a secondary layout used by resolver which does not
contain R.id.empty. Check for null.

Bug: 133707284
Test: Open PDF file through resolver
Change-Id: I311f1c1d2fa6edcebc81d9056755884b1585a7cb