Merge "Fix non-unique PendingIntent issue with TCImpl." into pi-dev

Cleanup documentation for LE CoC in BluetoothAdapter

Test: Compile
Bug: 77631591
Change-Id: Ic3c7f13e560534a048bf5c8b274fe62190c214c7
(cherry picked from commit dac8e140184f3ac52210ff3638af3f53edca1ba0)

Merge "Clarify API doc for BluetoothGattServer::addService" into pi-dev

Merge "AAPT2: Support id reference chaining from AAPT" into pi-dev

Merge "[DO NOT MERGE] [Companion] Call notifyDataSetChanged on main thread" into pi-dev

Merge "Fix channels --stats dump" into pi-dev

Merge "Fix screen on count text dump" into pi-dev

Merge "Don't crash during layout of non-attached but focused view" into pi-dev

Fix channels --stats dump

Test: manual inspection
Bug: 78292990
Change-Id: Iab75627494213870184e503bc2585cc8644ca1dd

Fix non-unique PendingIntent issue with TCImpl.

As per the referenced bug, we're running into issues where apps are
being fired with stale intents. The reason is because we need intents we
fire to be unique by Intent.filterEquals. Some of the intents we
generate put unique data in the intent extra which is not considered by
filterEquals. The solution here is to create PendingIntents with unique
request codes (using classifiedText.hashCode()).
See more info about this in
https://developer.android.com/reference/android/app/PendingIntent.html

Bug: 77930684
Test: manually tested broken scenarios. See referenced bug
Test: bit FrameworksCoreTests:android.view.textclassifier.TextClassificationManagerTest
Test: bit CtsViewTestCases:android.view.textclassifier.cts.TextClassificationManagerTest
Test: bit FrameworksCoreTests:android.view.textclassifier.TextClassificationTest
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Change-Id: Ib7275f94ca5ada51e4ba191742d4b614df12e1ea

Merge "Adjust media.codec process group upon request from mediaserver" into pi-dev

Merge "SyncManager: Dump per-source last sync time too." into pi-dev

Merge "Split provider / service dumpsys into platform and non-platform" into pi-dev

Merge "Update notification shade text on zen change" into pi-dev

Merge "getUserIdLPr should take application id as input" into pi-dev

Merge "Update bluetooth QS tile secondary text." into pi-dev

Merge "Fail gracefully if we get a bad API whitelist." into pi-dev

Merge "Don't mute ringer if apps can bypass dnd" into pi-dev

Merge "Add setMin for all slider in volume panel." into pi-dev

Merge "Clean up punctuation in SIM locked strings." into pi-dev

Merge "Status bar icon ordering" into pi-dev

Merge "Translate SystemUi NotifChannels on locale change" into pi-dev

Merge "Update docs 'since' for API 28." into pi-dev

Merge "Revert "Don't consider resources < 0 as invalid"" into pi-dev

Merge "Visit Uris in RemoteViews for granting purposes." into pi-dev

Update notification shade text on zen change

Change-Id: I6b75af1a94f7da95ac78fac122651924db7c1308
Fixes: 76143428
Test: NotificationStackScrollLayoutTest

Clean up punctuation in SIM locked strings.

Bug: 76122305
Test: visual
Change-Id: I4795b10fee950fd37efc0bd95af87cb14d0947ff

Translate SystemUi NotifChannels on locale change

Register receiver outside of the on boot receiver

Test: manual
Change-Id: Id286647ffe68704f928ca55dcae14dadc4fa998d
Fixes: 64887866

Merge "AOD: Ensure WakeLock is held for delayed dozeTimeTick" into pi-dev

Update bluetooth QS tile secondary text.

Only show detailed status when connected to exactly one device.
Otherwise show "X devices".
Also, listen for bluetooth profile service callbacks to initialize the
bluetooth device state after systemui crashes.

TODO: Update string to "X devices connected" after P

Change-Id: I99a3484c3dfa7eb0d6d8836336ebb84cbdaadbe5
Fixes: 76102598
Fixes: 77498698
Test: manual

getUserIdLPr should take application id as input

Fixes: 78265824
Test: Setup work profile using TestDPC -> add account into work profile.
Test: Add google.com in personal side to trigger work profile setup.

Change-Id: Ib87301fc98d0ebf1baa043e4badb5144a0f62377

Revert "Don't consider resources < 0 as invalid"

This reverts commit fbcf20819182107fcf1d3fb894881754e2fbb7f6.

Reason for revert: Breaks all window animations

Bug: 78264055
Change-Id: Iaa09626ad8dc26a3fde1ecc402dc2bd28df99627

Update docs 'since' for API 28.

I'm replacing this list with a for-loop in master, so it shouldn't need
updating in future (except for temporary apis).

Bug: 77588754
Test: make docs
Change-Id: Ib48218e303b53e2b16a13866e4423ee2ad1c2f4a

Merge "Animate home in separate layer" into pi-dev

Merge "Add one more hidden API to light greylist" into pi-dev

Merge "Add UUID->leastSigBits to light grey." into pi-dev

Add one more hidden API to light greylist

Bug: 78229662
Test: make
Change-Id: I603e48d795d6d805f46389f9edbd81ffd6c66696

Fail gracefully if we get a bad API whitelist.

If we send a bad API whitelist to the Zygote, it causes it to close the
socket. If we take no further action in AMS, it results in the same list
of exceptions being sent when we re-open the socket, resulting in it again
being closed. This results in no longer fork/start any new processes.
Since the list is persisted, this would result in the device entering a
boot loop upon reboot. Since no apps could be started, we cannot recover.

So in the case that the exemptions list causes problems, clear out the
list so we don't try to send it again next time. This means we will see
a single failure, but future attempts will succeed (obviously without
any whitelist). The device should not enter a boot loop.

Note, the test below relies on the fact that we can send at most 1024
arguments in a command to the Zygote (MAX_ZYGOTE_ARGC), and that each
item on the list is a separate argument.

Test: adb shell settings put global hidden_api_blacklist_exemptions \
Test:    $(for i in {1..1025}; do echo -n $i,; done)
Bug: 64382372
Change-Id: Ie47095d516c247ff6a8d667a2ac9b7be45f1acda

Merge "Add proto constants for hidden API logging." into pi-dev

Merge "Fix notification bar can be dragged when SIM PIN lock screen." into pi-dev

Merge "Fix unnecessarily extended View lifetime in IMMS" into pi-dev

Visit Uris in RemoteViews for granting purposes.

RemoteViews end up passing around Uris, so we need to extend Uri
permission grants for them to ensure the recipient of a Notification
object is able to render its contents.

Bug: 9069730
Test: atest frameworks/base/services/tests/uiservicestests/src/com/android/server/notification
Change-Id: Id31b5adaf2ee66113a1b503e32126aeddbf97b28

Merge "Fixed autofill forged package check to only compare the package name." into pi-dev

Fixed autofill forged package check to only compare the package name.

Otherwise, it will log false negatives when app launches a new activity in the
same autofill session. Example:

04-18 19:34:32.392  5423  7199 W AutofillSession: Activity ComponentInfo{com.netgear.WiFiAnalytics/com.netgear.WiFiAnalytics.WiFiAnalyticsWifiRoomSignal} forged different component on AssistStructure: ComponentInfo{com.netgear.WiFiAnalytics/com.netgear.WiFiAnalytics.WiFiAnalyticsWifiAnalyzerTab}

Fixes: 78235438
Bug: 69981710
Test: manual verification, cannot be CTS tested.
Test: CtsAutoFillServiceTestCases

Change-Id: I0408cd159c2be22841a1f6c36a4a2c17e59a6219

Merge "Implement swipe up and quick scrub onboarding" into pi-dev

Merge "Reducing the touch slop for quickstep" into pi-dev

Fix unnecessarily extended View lifetime in IMMS

Currently InputMethodManagerService (IMMS) unnecessarily extends the
lifetime of View objects used to show IME switcher dialog until the
dialog is shown again, because it forgets to clear
IMMS#mSwitchingDialogTitleView with null even after the dialog is
closed.

As a result, roughly 1.5MB of RAM will be unnecessarily consumed by
the system_server once after the IME switcher dialog is shown until
the device is rebooted.

With this CL, IMMS#mSwitchingDialogTitleView will be cleared with null
when the dialog is closed.

Fixes: 78245299
Test: Manually made sure that IMMS does not hold strong references to
      View objects used for IME switcher dialog after the dialog is
      closed.
Change-Id: I9fee92bb2974edeceacc680686095df480efba29

Merge "No AppStandby tracking of non existent packages and add parole window" into pi-dev

Merge "Added more documentation for dialogMessage" into pi-dev

Implement swipe up and quick scrub onboarding

Bug: 70180942
Test: manual test
Change-Id: I5c487d1e4ab800450c6b5a8f7ad1bf7434e45904

Merge "Rely on tile state for on/off a11y announcement." into pi-dev

Add setMin for all slider in volume panel.

As not only accessibility stream have a min volume, setMin for every
stream to avoid mismatching between volume panel and actual volume.

Bug: 72403580
Test: Add log in onProgressChanged and check min value.
Change-Id: Ief94faeeaa5ca929d0b00ee197eb12fddea4c3b5

Merge "Switch user only if the user is changed" into pi-dev

Merge "ActivityManagerService: add restricted cpuset" into pi-dev

Merge "Make Instrumentation more robust to other inputs" into pi-dev

Reducing the touch slop for quickstep

Also fxing some wrong thresholds being used for slop comparisions

Test: Verified on device
Bug: 78232923
Change-Id: I0b10a921002f2e45d0d2472a3e690d1526f4aecb

Merge "Ignore mkdirs requests if CE storage not available." into pi-dev

Clarify API doc for BluetoothGattServer::addService

Clarify that the caller has to wait for onServiceAdded callback before
calling BluetoothGattServer::addService again.

Bug: 72717069
Test: Compile
Change-Id: I20b031c724ba64bfd71cf10e58e587f69e4a2555
(cherry picked from commit 4b5cf4856064efbc018c0f2b4d6d3ff199403ee0)

Added more documentation for dialogMessage

Added explanation that callers can use a format string which takes a
single argument which is the name of the suspended app that the user
tried to launch.

Test: make docs

Bug: 77507744
Change-Id: I0a5259048332030385265ceab9c7d76766abac7d

Merge "Dump IOmxStore" into pi-dev

Don't crash during layout of non-attached but focused view

Verifies that there is a viewrootimpl before calling its
methods.

Bug: 77193567
Test: N/A
Change-Id: I301c3aede3d0f7e1d2abf6fe5bb3d852f697577b

No AppStandby tracking of non existent packages and add parole window

Disallow attempts to change standby bucket of nonexistent packages.
Also, add a parole window for idle apps. When parole for idle apps is
ready to be set, check that there is network availabilty. If there is
none, wait for network or the end of the parole window before setting
parole.

Change-Id: I7b8786f59b8da76325f2591cc7b1f86753d52d7b
Fixes: b/71857185
Fixes: b/74345840
Test: atest android.app.usage.cts.UsageStatsTest#testIgnoreNonexistentPackage
Test: manual (enable DEBUG and COMPRESS_TIME in AppStandbyController)

[DO NOT MERGE] [Companion] Call notifyDataSetChanged on main thread

Fixes: 78090775
Test: ensure attached bug not reproduces
Change-Id: I386087742883e4847d811f7b8e6e7c8c8aea227a

Merge "AAPT2: Modified StringPool uniqueness detection #2" into pi-dev

Merge "Set default reference ID" into pi-dev

Merge "Add setMin for A11y stream slider bar." into pi-dev

AAPT2: Modified StringPool uniqueness detection #2

b/77862560 detected that when converting an apk to binary using aapt2,
all resource ids of attributes that have been replaced with resource
identifiers become set to the identifier of the first attribute. This is
because the attribute names are all empty because the names are not necessary
since the resource ids are present. The empty attribute names all map to
the same string pool reference and cause all the ids to be the first
empty string into the string pool.

The ag/3897499 approach to fix the specified bug was extremely inefficient
and caused hour long builds. This change takes advantage of the multimap
data structure to do lookups efficiently.

Bug: 77862560
Test: Converted apk in listed bug from proto to binary and observed
correct resource ids and correct badging. Also built the Android tree to
check for regressions in build time.

Change-Id: I27a9ee4ffbed8b9ff6f238ad315cdf87b588947c

Merge "add dump report reason to reports" into pi-dev

Adjust media.codec process group upon request from mediaserver

Add method to allow dynamic adjustment of cpuset of media.codec
process. Only accept requests coming from mediaserver.

Bug: 72841545

Change-Id: Idb09d9a5162691503ecf6d811a528d9160326358

add dump report reason to reports

+ also change uidmapping version numbers to int64_t

Bug: 78132855
Change-Id: Iac7ea93e4bf651bd65bd03383e7ab4971af4fc29
Fix: 78132855
Test: gts test

Merge "Add carrier config for partial sms raw message expiration." into pi-dev

Make Instrumentation more robust to other inputs

Test: atest cts/tests/app/src/android/app/cts/InstrumentationTest.java
Change-Id: I2d55274ab19c4687ce85c086f2da61a4316b8096
Fixes: 70720100

Merge "Prevent early onEnterAnimationComplete callback for recents target" into pi-dev

Merge "Fix regression in assist data fetching and canceling" into pi-dev

Merge "Use config_longPressVibePattern as heavy click fallback." into pi-dev

Add setMin for A11y stream slider bar.

Since the volume of accessiblity will not go to 0, add setMin for A11y
stream slider in case it goes to 0 in volume panel.

Bug: 72403580
Test: Add log in onProgressChanged and check A11y stream won't go to 0.
Change-Id: Ie8c88c8a40fad4d281997922df8acb8f446288c7

Merge "Ensure QS does not expand when qs disabled." into pi-dev

Merge "Serialize key eviction vs. user storage preparation" into pi-dev

Merge "Record timestamp when brightness tracker is called" into pi-dev

Merge "ViewRoot: Ensure we release surface from setWindowStopped." into pi-dev

Merge "Don't apply corner cutout insets if we're already inset enough" into pi-dev

Merge "ActivityManagerService: ensure updateOomAdjLocked gets called early" into pi-dev

AAPT2: Support id reference chaining from AAPT

AAPT would allow for ids to be declared in the form:

<item name="name" type="id>@id/other</item>

@id/name should hold a reference to @id/other. When
getResources().getValue() is called on R.id.name with resolveRefs
enabled, the resuling reference should be R.id.other.

Bug: 69445910
Test: Created tests for correct parsing of id references and correct
resolving of deep references

Change-Id: Id1feb37b2565c213dc6a19b4c401906260d7fc14

Status bar icon ordering

Hotspot > DnD > volume

Test: visual
Bug: 74985733
Change-Id: I70adb3300dc842bba47283082035ce1a69de5b1d

Merge "Don't consider resources < 0 as invalid" into pi-dev

Merge "Grant notification Uri permissions as sending app." into pi-dev

Merge "Add AccountManager key rotation awareness." into pi-dev

Don't mute ringer if apps can bypass dnd

Test: atest RankingHelperTest
Change-Id: I587e5b8d4f6cfe3b384c5b4b9e72de12b6639b6f
Fixes: 77729075

Merge "Add setting for GNSS location request update duration" into pi-dev

Record timestamp when brightness tracker is called

Associate the brightness slider event timestamp with
the time the tracker is called rather than when
the background handler handles the call.

Bug: 78224086
Test: atest BrightnessTrackerTest
Change-Id: I083fdcc10f223ab7398911d9cb63dca5d8681fc4

Merge "Revert "Remove the OEM hook implementation and usage"" into pi-dev

Use config_longPressVibePattern as heavy click fallback.

Rather than using the regular click effect as the fallback, use the long
press pattern since heavy click is the new effect for long press. This
also lets us distinguish the two effects in the configuration.

Bug: 77863933
Test: long-press on something, feel the effect
Change-Id: Ie22d064cbdd2d33702180cb528d743e75ff8ae63

Don't consider resources < 0 as invalid

The only invalid resource ID is '0'. All other resource IDs are
valid; even negative resource IDs.

With the introduction of namespaces in AAPT2, resource IDs start with
0x80, 0x81, ... [ie. because Java only supports signed types, they are
considered negative]. For app transition animations negative resource
IDs were incorrectly considered "invalid".

Change-Id: I24032baa54860459d4f1b8e17a80c760c48d5579
Fixes: 70716301
Test: Manual. Run multi-split APK and see that transition animations work when defined in a split

Merge "Fixes no vibration on old nav bar" into pi-dev

Set default reference ID

The reference ID may not be set [when the reference ID is 0x00], so,
we need to ensure it has a proper default value; instead of garbage

Change-Id: Ie1bcf062fbb1aab5832cf210f68b24732af5dc3b
Fixes: 74389717
Test: Run gReader and see that it doesn't crash

Merge "Revert "Fix "Screen is pinned" dialog not showing completely in multi-window"" into pi-dev

Revert "Fix "Screen is pinned" dialog not showing completely in multi-window"

This reverts commit be26707695e3cf4785e716b9814a067d50cd1123.

Reason for revert: Caused regression in b/77976945

Fixes: 77976945
Bug: 73513388

Change-Id: I8c83a823d7421563ea5a2eab2e20cb2b06ec6068

Merge "Fix issue #74248644: Swiping away task increments start ID of running services" into pi-dev

Merge "Fix merge isolated uid to host uid" into pi-dev

Rely on tile state for on/off a11y announcement.

Do not include on/off in QS tile content descriptions.

Change-Id: I10ddeb6c56f99fe4dad3eaaa98a56cafba68be9f
Fixes: 78171152
Test: manual