Merge "End the TC session on terminal selection event actions" into pi-dev

Merge "Fix slider interaction collection." into pi-dev

Merge "Unify behavior of various cases of "no underlying networks"" into pi-dev

Merge "Fix crash after smart selection animation" into pi-dev

End the TC session on terminal selection event actions

This regressed when introducing TC sessions in
I3c9ceea0863099fc4f0a5ce5e823c648ee9c4521
When the user triggers a terminal selection event such as "Copy",
we should immediately end the session instead of waiting for the
"Abandon" event (i.e. selection dismissed) to be included in the
logs. Terminal selection events implicitly dismiss a selection and
we'd rather distiguish between an actual "selection dismiss" from
one that happened because of a "terminal" selection event.

This cl also removes the "*" marker used to distinguish the new
logging from the old ones. The code for the old logging has already
been deleted so no more need for a marker.

Bug: 78541105
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Change-Id: Iac7d45dbc63e7076683742bd045766a1d927cfc9

Merge "Reduce LoadedApkAssets LRU cache size." into pi-dev

Merge changes Ic8231b18,I30009f88 into pi-dev

* changes:
  Fix: VPNs update caps upon underlying network disconnect.
  Add tests for setUnderlyingNetworks.

Fix slider interaction collection.

Brightness changes were always considered temporary
so we skipped notifying the tracker.

Bug: 79990855
Test: atest android.display.cts.BrightnessTest
Change-Id: I5702694218c9811f9f1413d8aed3cb57063ea1c8

Merge "Check settings validators for NPE" into pi-dev

Reduce LoadedApkAssets LRU cache size.

The cached apk assets can take up multiple megabytes of .apk mmap and
native memory for associated data structures. Reduce the cache size
for potential multi-megabyte memory savings with hopefully no
noticeable performance impacts.

In particular, rough estimates of the memory and performance impacts
of this change were evaluated by starting the launcher app,
sampling system server PSS with dumpsys meminfo and cache stats using
dumpsys activity for a few different cache sizes:

LRU Size   1: cached apks: total= 1 created=0 evicted=33 hit= 0 miss=34 max=1    TOTAL PSS=187542
LRU Size   3: cached apks: total= 3 created=0 evicted=12 hit=19 miss=15 max=3    TOTAL PSS=190606
LRU Size  15: cached apks: total=15 created=0 evicted= 0 hit=19 miss=15 max=15   TOTAL PSS=199350
LRU Size 100: cached apks: total=15 created=0 evicted= 0 hit=19 miss=15 max=100  TOTAL PSS=201416

Comparing cache stats after launching the settings app:

LRU Size   3: cached apks: total=3 created=0 evicted=5 hit=14 miss=8 max=3
LRU Size  15: cached apks: total=6 created=0 evicted=0 hit=18 miss=6 max=15

Running performance tests locally to ensure no clear regression in app
startup:

app-startup/benchmark-app-hermetic/cold-dropcache-test:
LRU                15        3        3
activityStart:    40.47    38.53    39.27
bindApplication:  24.40    24.53    24.93
draw:              4.27     4.33     4.07
layout:            0.73     1.07     0.73
onCreate:          9.40     9.00     8.53
totalLaunchTime: 183.13   183.27   187.07

app-startup/hermetic-apps/cold-dropcache-test:

LRU                   15       15          3         3
Calculator_avg:      333.9    333.1      343.3      334.6
Calculator_max:      345.0    360.0      370.0      363.0
Calculator_median:   333.5    331.5      343.5      333.5
Calculator_min:      320.0    317.0      320.0      311.0
Calculator_std_dev:    6.62    13.26      18.22      16.51
Clock_avg:           510.0    511.2      509.2      512.6
Clock_max:           529.0    528.0      530.0      529.0
Clock_median:        510.5    509.0      503.5      512.0
Clock_min:           498.0    495.0      487.0      492.0
Clock_std_dev:         7.70    11.71      15.25      11.07
Contacts_avg:        457.7    527.3      507.3      543.9
Contacts_max:        602.0    622.0      591.0      588.0
Contacts_median:     410.5    568.5      568.0      580.0
Contacts_min:        390.0    374.0      366.0      394.0
Contacts_std_dev:     85.64    96.80      88.80      72.97
Phone_avg:          6073.1    6109.7    6110.8     6115.8
Phone_max:          6196.0    6208.0    6180.0     6222.0
Phone_median:       6070.0    6117.5    6100.0     6133.5
Phone_min:          5982.0    5990.0    6061.0     5994.0
Phone_std_dev:        61.49     67.35     34.55      60.17
Settings_avg:        457.9     446.6     451.6      428.7
Settings_max:        487.0     468.0     494.0      449.0
Settings_median:     460.0     450.0     444.5      424.0
Settings_min:        423.0     421.0     431.0      420.0
Settings_std_dev:     21.21     12.02     20.46       9.71

Bug: 79115008
Test: Device boots.
Test: dumpsys activity all shows ResourcesManager max cache size of 3.

Change-Id: I7225b6978fac6197c27bec23d2b477fb1f5aee43

Unify behavior of various cases of "no underlying networks"

Before this change, VPNs having no underlying networks would be
marked as metered as the safe option, but VPNs having only
disconnected underlying networks would be marked as unmetered.
Fix this discrepancy.

Bug: 79748782
Test: runtest frameworks-net
Change-Id: I51c3badde29f43f692f383553bd98327d2da8dd1

Fix: VPNs update caps upon underlying network disconnect.

Bug: 79748782
Test: ConnectivityServiceTests still pass
Change-Id: Ic8231b18a17e6feb5ebafe8d5688fb59f9d4d58e

Fix crash after smart selection animation

At the end of the smart selection animation, we run a callback that sets
the selection on the TextView (subsequently starting the action mode
toolbar and showing the handles). However, when the text changes before
the animation finishes, the selection might not be valid, and might get
out of the text bounds, which was producing a crash. This was observed
in a monkey crash. This CL fixes this bug by refusing to set the
selection when it goes outside the text bounds, corresponding to the
case when text has changed between the time the animation has started
and the time it ended.

Bug: 69919777
Test: manual testing before and after the fix
Change-Id: Iea043f320004d45ad16dd7e9e5b47e5256e6d9fa
(cherry picked from commit cce6e82d35b5a6c8eb29e76fbae53eae8b70e99a)
Merged-in: Iea043f320004d45ad16dd7e9e5b47e5256e6d9fa

Merge "Add more hidden API usages into greylist" into pi-dev

Add tests for setUnderlyingNetworks.

Fixes come later. This is complex enough as it is.

Bug: 79748782
Test: new test passes, old tests still pass
Change-Id: I30009f88e68a534c332ca88bae517cacc39a60bb

Merge "Move CaptivePortalProbeResult to its own file" into pi-dev

Merge "Fix setCapabilities." into pi-dev

Merge "Add new hidden-for-now AppOpsManager APIs to greylist." into pi-dev

Clock notification text was cut off

The root cause is that HeadsUpStatusBarView doesn't considerate the
Cut Out situation. The Cut Out situation can use
getRootWindowInsets().getDisplayCutout().getSafeInsetLeft() to get
the the value.

There are two parts need to handle Cut out.
The one part is to handle the padding. It needs to considerate both of
mLeftInset and mLeftCutOutInset because it use getLocationOnScreen to
count the location.

The other part is to handle the HeadsUpStatusBarView.translationX.
It only needs to considerate mLeftCutOutInset because landscape
degree 90 has the left side cut out and translationX by the distance
between screen left boundary and scroller's left boundary. The
distance include Cut Out so it need minus mCutOutInsetLeft in the
setPanelTranslation.

Cut Out has 4 mode: Disable, Corner, Double, and Tall. Disable and
Double are handled by the same way. Corner and Tall are handled
by the same way.

Bug: 78113562
Test: atest SystemUITests
Change-Id: Ic2a272c43f65eed8c4b3749787637f5fb848bb8a
Fix: 78113562

Merge "Add content description to zenmode" into pi-dev

Merge "Bugfix: PIN keyguard is clipped in Car" into pi-dev

Merge "Show swipe-up tip with exponential backoff" into pi-dev

Show swipe-up tip with exponential backoff

Change-Id: Icd32c0bd98623173322e5e33bd5400e679a01a10
Fixes: 79985309
Test: Manual test

Bugfix: PIN keyguard is clipped in Car

  Car is a multi-user system.  There's a cancel button on the Keyguard
  that allows the user to cancel the Keyguard to go back to user
  switcher.  Because different user may have different security mode,
  the container must be resized when the security mode changes.  The fix
  is to destroy the old bouncer view on cancel.

  Screenshot of clipped view: https://screenshot.googleplex.com/0VWdDtrLkAr

Fix: 79202566
Test: In the head unit
Change-Id: Ife33c53785ed96778f08c8f5886885e2b9d29749

Move CaptivePortalProbeResult to its own file

This is necessary to resolve visibility issues for the next change.

Bug: b/79499239
Test: runtest frameworks-net
Change-Id: I50bc96afe6ae88c8f58a693f0a4e821f1f9b3299

Merge "Bouncer shouldn't be translated when occluded" into pi-dev

Add content description to zenmode

Test: manual
Change-Id: Ic478e8a6fcd2088d198b2cee2e70ab8376b4709d
Fixes: 79686343

Merge "Measure text in 1 pass" into pi-dev

Merge "Differentiate MediaController in a process for MediaSession" into pi-dev

Bouncer shouldn't be translated when occluded

Only checking if the bouncer should be scrimmed isn't enough
since it might have been initialized for performance reasons,
but is hidden at the bottom of the keyguard.

Change-Id: If92083e2c385d51febe0ec3a5e5d3484476caf81
Fixes: 80101176
Test: atest packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/StatusBarKeyguardViewManagerTest.java
Test: manual

Merge "Add utility methods to AmbientDisplayConfig" into pi-dev

Merge "Pulse state should always show ambient wallpaper" into pi-dev

Merge "FingerprintService: requires ro.product.first_api_level" into pi-dev

Merge "Setup mAreChannelsBypassingDnd from notifPolicy" into pi-dev

Add new hidden-for-now AppOpsManager APIs to greylist.

Bug: 78480444
Test: manual
Change-Id: I2a9c2040279df3d875b96edb4a686006f5114a76
Merged-In: I2a9c2040279df3d875b96edb4a686006f5114a76

Measure text in 1 pass

Before we were setting the text again using setText, and this
would trigger a second measure/layout pass, which is unnecessary.

Now we're using the same mesure pass to calculate the view's
best line break.

Fixes: 79714443
Test: manual. set various meeting names animated or static
Change-Id: Ia911fae8f820b703ca385a10211073bae37f3258

FingerprintService: requires ro.product.first_api_level

Test: fingerprint registration works
Bug: 79950359
Change-Id: I1c75b9258a5c63b23fa2c546f002a596c9d55dc3

Pulse state should always show ambient wallpaper

Otherwise wallpaper wouldn't be set to AOD when the device pulses
and always on is off.

Bug: 78606979
Test: manual
Test: atest packages/SystemUI/tests/src/com/android/systemui/doze/DozeWallpaperStateTest.java
Change-Id: Ic40c18252cd6cb06ff6fd0d79ec3f1de16a1add5

Setup mAreChannelsBypassingDnd from notifPolicy

Test: atest RankingHelperTest
Change-Id: I34d9c8dfb5924bab0e496e37d5b93715c48b78ad
Fixes: 80078295

Merge "DO NOT MERGE - Switch user picker/switcher to use CarUserManagerHelper." into pi-dev

Merge "Fix the Hearing Aids connected state in Settings App" into pi-dev

Add utility methods to AmbientDisplayConfig

The added utility methods are to enable increased
granularity in determinining the availability of gesture
settings dependant on Always on Display.

Bug: 79779562
Test: robotests
Change-Id: Ib54a867ca554d44ee4190706805fa8545aec9fa0

Merge "Extend rotation button timeout duration" into pi-dev

Merge "Allow UiAutomation being used off the main thread" into pi-dev

Merge "Remove MANAGE_USERS from setPackagesSuspended" into pi-dev

Merge "Make RemoteCallback test API" into pi-dev

Merge "Check for bpf support at NetworkStatsService start" into pi-dev

Merge "Keep scrim black when "always on" is off" into pi-dev

Extend rotation button timeout duration

Moving from 6s to 10s timeout when navbar is visible.

Change-Id: I3a7067c37f254698ae53fdb9208576f3e486c8ce
Fixes: 79981820
Test: manual

DO NOT MERGE - Switch user picker/switcher to use CarUserManagerHelper.

Test: Build
Bug: 79417730
Change-Id: I52c9b0f5ba01b85e36242b112ef9bbcc54a08e48

Merge "Add a shell command for IMMS to test instant apps" into pi-dev

Merge "Add a field in config to disable/enable the string hashing in metric report." into pi-dev

Merge "ResXMLTree: Clone DynamicRefTable on creation" into pi-dev

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

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

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

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

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

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

Merge changes from topic "am-d326b840-2575-4e69-8ec2-5cfa14046328" into oc-dev am: 99e6aa1b1e am: d88c6877da
am: aa55d5db76

Change-Id: If6e65aa957489e085d1a96be0367e158c43ebc5b

[automerger skipped] Import translations. DO NOT MERGE skipped: f72be588eb skipped: ea0127f477 am: 4f1f428661  -s ours am: a53d91e485  -s ours
am: 5e1f17a880  -s ours

Change-Id: I5fdb579286d6ddce6f6a20b2fab243b33840bf93

Merge changes from topic "am-d326b840-2575-4e69-8ec2-5cfa14046328" into oc-dev am: 99e6aa1b1e
am: d88c6877da

Change-Id: I0acd60a59fcb928752ce7bf952bbfe9f65b68ff4

[automerger skipped] Import translations. DO NOT MERGE skipped: f72be588eb skipped: ea0127f477 am: 4f1f428661  -s ours
am: a53d91e485  -s ours

Change-Id: Ife13f7c8262521ddc28deb4ceaf5b27b9765c792

Fix the Hearing Aids connected state in Settings App

When two Hearing Aids devices are bonded, we should only show one device
in the Settings App and at the correct location based on the connected
state of the HA devices. When at least one HA device is connected, then
the Settings App should show it as connected. Otherwise, the Settings
App should show it as disconnected when both devices are disconnected.

Bug: 79760469
Test: Run the robotests for Settings and SettingsLib
Change-Id: I33e19af054b686b3d71b00dcbd6ff16febde6099

Merge changes from topic "am-d326b840-2575-4e69-8ec2-5cfa14046328" into oc-dev
am: 99e6aa1b1e

Change-Id: Id35ca83236112898b4e39b3cb173be70c435168f

[automerger skipped] Import translations. DO NOT MERGE skipped: f72be588eb skipped: ea0127f477
am: 4f1f428661  -s ours

Change-Id: Iac8c22f36859d6648ae856236c9a97080238cae5

Keep scrim black when "always on" is off

The first frame when waking up is expensive and we
need a black scrim hiding what was left behind.

Bug: 78022321
Test: atest packages/SystemUI/tests/src/com/android/systemui/statusbar/phone/ScrimControllerTest.java
Change-Id: I99e2b7960daf2cbe51bec6aee4b15c74d532078c

Merge "WifiConfiguration: New temp disable network status for no internet" into pi-dev

Merge changes from topic "am-d326b840-2575-4e69-8ec2-5cfa14046328" into oc-dev

* changes:
  [automerger skipped] Import translations. DO NOT MERGE skipped: f72be588eb skipped: ea0127f477
  [automerger skipped] Import translations. DO NOT MERGE skipped: f72be588eb
  Import translations. DO NOT MERGE

Add more hidden API usages into greylist

Bug: 78610453
Bug: 80075119
Test: make
Change-Id: Ia0c7825a5a8c78bb7cfb89e8e9eca902dff9cd25

Allow UiAutomation being used off the main thread

Test: cts-tradefed run cts -m CtsUiAutomationTestCases
      cts-tradefed run cts -m CtsAccessibilityServiceTestCases
      cts-tradefed run cts-instant -m CtsAccessibilityServiceTestCases

bug:80024164

Change-Id: I72c565d28f4cbcf6775535fb846b268b785a6b34

Add a shell command for IMMS to test instant apps

This change adds a shell command to allow InputMethodManagerService
(IMMS) to bind to InputMethodService provided by instant apps, like we
did so for AccessibilityManagerService [1].  Note that binding to an
InputMethodService provided by instant apps is not a supported
scenario.  In theory we can avoid doing this by having a separate APK
for MockIme instead of putting everything into
CtsInputMethodTestCases.apk but that configuration is not yet
supported by our test harness hence it doesn't work right now hence
having this special mode for testing is the only way we have right
now.

 [1]: Ifced735a9a6e495747372dd8b00fdd64933a09c7
      d223db316d11a625a73f86392e7055d5c6f26e7d

Bug: 79484568
Test: Manually verified as follows:
  1. Build a test IME that has android:targetSandboxVersion="2" in its
     AndroidManifest.xml.
  2. Install the test IME APK with 'adb install --instant <APK>'
  3. adb shell ime list -a -s
  4. Make sure that the test IME is not in the list.
  5. adb shell cmd input_method set-bind-instant-service-allowed true
  6. adb shell ime list -a -s
  7. Make sure that the test IME is now in the list.
  8. Select that test IME.
  9. Make sure that the test IME is working.
 10. adb shell cmd input_method set-bind-instant-service-allowed false
 11. Make sure that the test IME is no longer the current IME.
 12. adb shell ime list -a -s
 13. Make sure that the test IME is no longer in the list.
Test: cts-tradefed run cts-instant -m CtsInputMethodTestCases
Change-Id: I4383129fd9e229a849282e874aff5d4eef1f49f8

Make RemoteCallback test API

Test: manual

bug:79782915

Change-Id: I61343573428333c0d4a9ee2523c444753280186c

Check settings validators for NPE

From b/79910479, we need to have better checks on settings validators.

Catch NPE for various settings validators and add a @Nullable annotation
to try to prevent NPEs in new validators.

Bug: 80067772
Test: atest SettingsValidatorsTest
Change-Id: I9a5058e1bae1ba0dea084c0589aa3e4ef3a51836

Merge "Status bar icons 15 dp" into pi-dev

Merge "Allow user to change AiAi's permissions." into pi-dev

Fix setCapabilities.

P introduced setSSID, UIDs and unwanted capabilities.
None of these exhibit commutative behavior through combineCapabilities
because their semantics don't allow it. Therefore
NetworkRequest.setCapabilities() is badly broken around any of
these. Look at the comments in the new tests to realize the
extent of the damage.

Bug: 79748782
Test: new tests written, old tests pass
Change-Id: Ie46581bdaf9ecc2f14aab44788bbdb27a3fec8c1

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I99ed00eac259f67c341c1a82a547221bb7082e1f

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Id52e0cc164ebb1c62710dfc81b4bf70d231e08fb

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I11a29c0ab563be1a3a6bdf96fc82e719ce98e9c4

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: I5e753540af680ab9f8514066a9b164cd51a149ce

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Icd1296a26d4c15ad5886b8cf1a7e0d438253b1cd

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Ia968369bf14114aab85aa2aaa60050936585d5d6

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

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 64712476
Change-Id: Id6f7b7b77a60fc3d42cd583a6d7e77f75b6b6bc4

Add new metrics constant for Bluetooth A2DP hardware offload

Bug: 63932139
Bug: 79568680
Test: Settings app compiles without errors
Change-Id: I535f9cd819787f6c8d24facb5824dbf7472d5231
Merged-In: I535f9cd819787f6c8d24facb5824dbf7472d5231

Merge "IMPORTANCE_FOREGROUND_SERVICE process can access fingerprint" into pi-dev

Merge "Fade back button in and out tied with the overview/shelf (1/2)" into pi-dev

IMPORTANCE_FOREGROUND_SERVICE process can access fingerprint

We added a new priority IMPORTANCE_FOREGROUND_SERVICE
for foreground services and the services to  which the
system is bound fall into this bucket while before they
were falling into IMPORTANCE_FOREGROUND.

Test: manual

bug:79775649

Change-Id: I1626e78c5b7365985c5b4152637ab99aa0dea5f2

Merge "Update auto-rotate and print icons" into pi-dev

Check for bpf support at NetworkStatsService start

The current implementation check for bpf map existance whenever a
NetworkStats request comes in. The check is not efficient and may
require additional permission for threads. So a member variable is added
to NetworkStatsService to store the bpf configuration status and is
initialized when NetworkStatsService start.

Test: -m CtsNetTestCases -t android.net.cts.TrafficStatsTest
      -m CtsUsageStatsTestCases -t android.app.usage.cts.NetworkUsageStatsTest
Bug: 79994577

Change-Id: I872ae18fb3e9631a4e6c4d773fefbba32e60a03b

Merge "clone the visibility objects for the handler thread" into pi-dev

Merge "Reset wallpaper state during startup" into pi-dev

Status bar icons 15 dp

Shrink all status bar icons by 2dp. This means setting a scale factor on
anything that is a StatusBarIconView, and manually scaling down wifi and
mobile data groups.

Also fix the padding between the mobile data view and the battery

Also snuck in a fix for the battery saver icon clipping on QS

Change-Id: I5f5430d9e9c66f5c2794dcf4974cf18aa3bff061
Test: visual
Fixes: 74985733
Fixes: 79967193
Bug: 77822905

Merge "Set enabledSurroundFormats as empty string when it is null." into pi-dev

Differentiate MediaController in a process for MediaSession

MediaSession#getCurrentControllerInfo() can have more fine grained
differentiation for multiple MediaController in a package/process

Bug: 79728675
Test: Run CtsMediaTestCases
Change-Id: I81e94736e795ee5e7fbd28cb6cd834cc3a3b8f98

Merge changes Id4feeefa,Ie0a501b8 into pi-dev

* changes:
  Add counter for predictive info for hush gesture
  Update DND strings

Merge "Make ImportWrappedKey work with real hardware:   Get unwrapping params from WrappedKeyEntry" into pi-dev

Update auto-rotate and print icons

Bug: 77982107
Test: manual
Change-Id: I7c5183c7e12819204ce5ecda1017b968ef4a9957

Merge "Autofill: fix missing focus/ripple effect for authentication view" into pi-dev