Merge "Adding some helper methods in QuickstepContract"

Merge "Proper package icon for Q SystemUI."

Merge "Ensure input transactions are applied before injecting input"

Merge "Fix am start -S <package>"

Merge changes from topic "rro-cts"

* changes:
  RRO CTS Test APIs
  Relax restrictions on non-system overlays

Merge "Fix idmap2 policy concatenation"

Fix am start -S <package>

Previously, am start <package> and am start -S <package> had different
behaviors because the -S code path did not convert the default user id
(by calling handleIncomingUser) into a proper userId that is uses for
querying intent activities.
The default user id is converted in other code paths, so we do not
convert it inside of makeIntent, but only in the mStopOption path, right
before queryIntentActivities.

Test: adb shell am start -S com.android.chrome
Test: adb shell run-as com.example.zhin am start
(same as P)
IllegalArgumentException: No intent supplied
Test: adb shell run-as com.example.zhin am start com.example.zhin
(same as P)
Security exception: Permission Denial: startActivityAsUser asks to run as user
-2 but is calling from uid u0a193; this requires
android.permission.INTERACT_ACROSS_USERS_FULL
Test: adb shell run-as com.example.zhin am start --user 0 com.example.zhin
(same as P)
Starting: Intent { act=android.intent.action.MAIN
cat=[android.intent.category.LAUNCHER] pkg=com.example.zhin }
Error: Activity not started, unable to resolve Intent {
    act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER]
    flg=0x10000000 pkg=com.example.zhin }
Bug: 129712542
Change-Id: I3ce6ffd40fd834bffd28b4440c1fb0024c799d95

Merge "Add onPointerDownOutsideFocus for events outside the focused window (4/4)"

Merge "PlayerBase: conditional muting on OP_PLAY_AUDIO changes"

Merge "Straighten AIDL interface for the memory store"

Merge "Adds the flyout view."

Merge "Using input monitor for edge swipe handling for back"

Merge "Add package name when loadModules and attach."

Merge "Removes unnecessary call to prepareScanResultLocked"

Merge "Set packages.list context to packages_list_file."

Merge "Add onDestroyView to allow clock plugins to clean up resources."

Merge "Set current user to USER_NULL when HAL dies"

Merge "Don't invoke resetLockout when not enrolled"

Adds the flyout view.

This moves the view itself from BubbleView to BubbleStackView, since there will never be multiple flyouts and it simplifies layout. This also adds getUpdateMessage to NotificationEntry which is used to generate the flyout text.

Test: atest SystemUITests
Change-Id: Ief2fcfb2b12b927fdd68f737d49080335c884bef

Merge "Disallow PIP for Volume Panel"

Merge "Remove focusable attribute from car_preference."

Merge "Ensure that we always log updating from file."

Adding some helper methods in QuickstepContract

Test: N/A
Change-Id: I3d4c05820659a1f91d4a23f2bfd2ffc56ed06234

Merge "Allow companion apps to start background activities from PendingIntents (even if they aren't foreground)"

Merge "Remove obscure check that was always true"

Merge "Merge "Clean up permissions when uids are no longer used" am: 7754d368f4 am: ecb28c1948 am: 16ee7a1be4"

Merge "Clean up permissions when uids are no longer used" am: 7754d368f4 am: ecb28c1948
am: 16ee7a1be4

Change-Id: I7773132d11d6225eac91ab2fee1043159c295d83

Fix idmap2 policy concatenation

When printing warning messages while generating idmaps, put the vertical
bar chracater in between the policies.

Bug: none
Test: manual

Change-Id: I8efee753d9ed7ce99ccafd49b6c8744eea031839

Merge "Add statically built regex pattern to avoid rebuilding all the time"

Merge "Snapshot arguments before processing."

Merge "Clean up permissions when uids are no longer used" am: 7754d368f4
am: ecb28c1948

Change-Id: I256ca6f16ea1fcd240a1b7cd2b3199d4504e6c33

Merge "OverlayInfo API: fix @Nullable -> @NonNull"

Merge "Avoid a bunch of log spam in SmartReplyConstantsTest."

Merge "Clean up permissions when uids are no longer used"
am: 7754d368f4

Change-Id: I3c06a36368abb9a1e7885d281ea1155fce857eea

Merge "Cancel Metric activations"

Merge "Increase char limit for face_error_not_enrolled"

Add onDestroyView to allow clock plugins to clean up resources.

Fixes: 129859743
Test: New test point added to KeyguardClockSwitchTest
Change-Id: Ia0eb7cb329820a64a4783937f65135fda742a2d1

Set packages.list context to packages_list_file.

Bug: 123186697

Test: adb shell su root ls -lZd /data/system/packages.list
-rw-r----- 1 system package_info u:object_r:packages_list_file:s0 35816 2019-04-02 14:17 /data/system/packages.list

Change-Id: I62ae03b5c2acf6c703f2e4c6f49cc68825e3a785

Using input monitor for edge swipe handling for back

Bug: 112934365
Bug: 124299674
Bug: 124298541
Test: Verified build on device
Change-Id: Ic4cc4339b22aee63b0d96587f05dae3d8c01563f

Merge "Dont sync keys using the unified challenge profile random credential"

Merge "Remove call identification APIs."

Merge "Clean up permissions when uids are no longer used"

Merge "Fix NPE in logActionClick"

Removes unnecessary call to prepareScanResultLocked

There were two calls to prepare scan result within a dozen lines of one
another. This was not resulting in any bad behavior, but was unnecessary
and resulted in unneeded warnings.

Test: atest CtsAtomicInstallTestCases:AtomicInstallTest
Bug: 109941548
Change-Id: I8817a189cb9976382a8057fe8b2c7a638f3d0d64

Merge "Enforce new bubble policy in NoMan"

Merge "Add metadata and headers to incident reports."

Merge "Change visibility of *WorkSourceUid APIs to public instead of @SystemApi."

Ensure that we always log updating from file.

If an exception was thrown by one of the methods, we were not logging
the event, which caused the CTS test to fail.

In particular, this happened on non-AB devices, such as the emulator.

Bug: 127268862
Change-Id: I8f853d5e0a8e810a570562071fb73266fee3b696
Test: atest CtsDevicePolicyManagerTestCases:com.android.cts.devicepolicy.DeviceOwnerTest#testInstallUpdateLogged

Merge "Change KernelCpuThreadReader frequency bucketing to work with >2 core clusters"

Merge "Don't lock the profile after setting work challenge."

Merge "Disable a flaky test, will fix later."

Merge "Ensure that THM sets up even when system dies"

Merge "libmedia2_jni: link to libcgrouprc"

Disable a flaky test, will fix later.

Test: Disabling a test
Change-Id: Ifc76fc8fc66ee57ebf87eedcad0e2474a19674da

Merge "Merge "Pass app's data dir to ART" am: 16749ac83e am: 33ed677361 am: 6277c39b0e"

Merge "Tweak bar chart measurement."

Merge "Pass app's data dir to ART" am: 16749ac83e am: 33ed677361
am: 6277c39b0e

Change-Id: Iec85a2630e364e66e45e42f2cbb6a26008c0ab4a

Merge "Support content description with app name"

Proper package icon for Q SystemUI.

Test: make, examine in Settings
Bug: 123903304
Change-Id: I81d8c283ed67725fa85c9a3116c7434cb00d3d22

Merge "Improve CursorWindow failure message"

Merge "Pass app's data dir to ART" am: 16749ac83e
am: 33ed677361

Change-Id: I9a7c617f13dadc91b735837d1778e3d526be8520

OverlayInfo API: fix @Nullable -> @NonNull

OverlayInfo#getTargetPackageName will never return null. Correct
@Nullable annotation to say @NonNull instead.

Fixes: 129853770
Test: builds, boots
Change-Id: I70a9634e4f7da99be5d9044a7884b9ad01a22fd8

Merge "Expose input region monitoring to system components."

Merge "Update the format of the am_compact event log message."

Improve CursorWindow failure message

Bug: 129139241
Bug: 129721058
Test: atest CursorWindowTest
Change-Id: Iac1c5ec6f999dadd638fc5ab47c69d13f60ea467

Merge "Adds the ability for SysUI to request  QSB alpha changes."

docs: Javadoc cleanup

There were a number of Javadoc style problems in the
WifiNetworkSuggestion.Builder.build() docs, which caused some formatting
problems.

One specific problem worth noting: A {{ causes trouble at publish time,
because it looks like the start of a Django directive. But escaping the
{ -- e.g. &#123 -- doesn't work, because the Javadoc compiler helpfully
escapes the *ampersand*, so the HTML output is "&amp;#123", which
renders on the page as "&#123" instead of "{". So putting a space
between the two braces seems like the simplest solution.

No code changes here, just Javadoc comment fixes.

Tested by building the docs and staging to:

http://go/dac-stage/reference/android/net/wifi/WifiNetworkSuggestion.Builder#build

Screenshots (in case the stage gets overwritten):
https://screenshot.googleplex.com/o4p7efFGCbJ.png (1)
https://screenshot.googleplex.com/nkhEqU54LNb.png (2)

Bug: 128536865
Test: make ds-docs (then build & stage)
Change-Id: Ie04bc4dc517443a6899aba989beeb38b9299826e

Merge "Remove user switcher when multiple users is disabled"

Merge "Don't announce changes in zen dialog"

Increase char limit for face_error_not_enrolled

Test: build
Fixes: 129708679
Change-Id: Id7ddb5a767dc33d701c2d8ba97672b41c93dccff

Merge "Pass app's data dir to ART"
am: 16749ac83e

Change-Id: I170cbff78d12ccc418cb440820a584f141f28433

Merge "Tidy up some Javadoc."

Merge "Set active stream when progerss changed by user"

Change KernelCpuThreadReader frequency bucketing to work with >2 core clusters

Change-Id: I470dc055b9b1eb07913280fc269fbdadb259b4db
Test: atest KernelCpuThreadReaderTest

Merge "Revert "Mute haptic channels by default.""

Merge "[Multi-user] Clean up user state stored in the system user directory when user is removed."

Merge "Pass app's data dir to ART"

Merge "Use a confirmedTap listener to guarantee a click. This eliminates the overlapping component listener bug from the caption service."

Merge "Use FLAG_BUBBLE rather than setting the value ourselves"

Allow companion apps to start background activities from
PendingIntents (even if they aren't foreground)

And push companion packages to ATMS after the user is
unlocked.

Bug: 129757565
Test: atest WmTests:ActivityStarterTests
Test: manual
Change-Id: Ic76d5c8a3fb096a8caf76dafb6c38212a8506f3a

Merge "OMS: ignore instant apps"

Merge "Change theme setting to JSONObject format."

Fix NPE in logActionClick

Problem:
The issue could happen when clicking on the archive button on the
notification from Gmail repeatedly and quickly. Tapping the Archive
button will change the notification layout to something custom
with a single UNDO button. The bug happens when the notification object
is already updated, while logActionClick of the old action button is
still running.

Proposed solution:
1. Add sanity check to avoid NPE, array index out of bounds.
2. To ensure that the action object is  the one we want, use PendingIntent in
Notification.Action as a token.

logActionClick is just used as logging, and the problem seems only happen
in repetitive clicks, so just bailing out should be fine.

Test: Have a Gmail notification, taps archive button repeatedly.
      Observe no crash.
Test: Ensure that log is sent in normal cases by checking logcat.

FIXES: 128804769

Change-Id: Ic04fd9288cba5253179bf1f478a7454e44f31703

Avoid a bunch of log spam in SmartReplyConstantsTest.

DeviceConfig logs errors when trying to read e.g. an integer and the
value provided isn't an integer. However, DeviceConfig avoids logging
the error if the value is null. So with this CL we use null as the reset
value for SmartReplyConstantsTest.

Bug: 129538548
Test: atest SmartReplyConstantsTest, and check logcat for spam.
Change-Id: I4c714b7e808163c20fccc5cdca734a255c698ede

Merge "CastTile: Better handling of multiple active devices."

Merge "Tell the app that a smart reply was used even when it has been edited."

Remove obscure check that was always true

This condition never made any sense because we checked the
systemUiVisibility field for a WindowManager flag. Instead of
trying to fix it, let's just remove the check to keep the
behavior.

Test: boots
Bug: 129751541
Change-Id: Ib0ace4d47f5623b32430a8a7e4f25f5a259451ea

Merge "Add more tests to KernelCpuThreadReaderDiffTest"

Tell the app that a smart reply was used even when it has been edited.

The existing API RemoteInput.Source covers two scenarios for responding
to message notifications:
1. A user uses the 'reply' button and enters a response freely.
2. A user clicks a smart reply which is used as the response.

Before this CL the platform would notify the app of scenario nr. 1 in
the case where we use tap-to-edit smart replies (replies that can be
edited before being sent). With this CL we instead log scenario nr. 2
(since a smart reply was clicked).

Bug: 123407240
Test: use a test-app to ensure that a response created using smart
replies + tap-to-edit logs the correct @Source.
Change-Id: I3dc8169a3c440af106c4f244b72ddf7c9e450203

Dont sync keys using the unified challenge profile random credential

When the work profile has a tied screen lock to its parent, its lock
credentials are set to a random password. This CL adds logic to prevent
syncing keys with this random credential.

On set/update lock:
- If creating the work profile or going from separate -> unified lock
screen: don't sync keys (random password case)
- If going from unified -> separate lock screen: sync keys
- If removing the parent lock: invalidate unified profile keys

On unlock:
- If unlocking a work profile with a unified lock: don't sync keys
(random password case).
- If unlocking a work profile with a separate lock: sync keys
- If unlocking a parent profile that has work profiles with a unified
lock: sync keys for the work profiles.

Design: https://docs.google.com/document/d/1y6LXcf-Rk3TMG-Ka4pJ5fpinDaK4fnlCyGi3kuGWWNg/edit?usp=sharing

Bug: 128834006
Test: 1) atest frameworks/base/services/tests/servicestests/src/com/android/server/locksettings/
2) Manual testing of the following cases by verifying key sync on the
backup device and being able to unencrypt the backup set on the restore
device:
a) Work profile unified lock screen: all 3 types (pin/password/pattern).
b) Changing parent lock screen in the unified case -> updates keys for
profile.
c) Unified lock screen -> separate lock screen: updates keys.
d) Separate lock screen and change credentials: updates keys.
e) Separate lock screen -> unified lock screen: does not update keys
with random password.
f) Unified lock screen -> remove lock screen: invalidates keys.

Change-Id: Ie2249f4c32fd6c48aae7f791e2d1e353b4ef9939

Change visibility of *WorkSourceUid APIs to public instead of @SystemApi.

Bug: 127465468
Test: atest BinderWorkSourceTest
Change-Id: Ib5743219ed44b893a64825d3d8c66e2f20b4197f

Merge "Use mPidMap instead of mProcessNames to get process with pid info"

Support content description with app name

The logic is like:
1. If content description string with app name placeholder is available,
   and we also have a app name, then we will use this string.
2. Otherwise, fallback to content description string without app
   name placeholder.

Test: atest frameworks/base/core/tests/coretests/src/android/view/textclassifier/
Test: Actually see the content description with app name is formatted
      properly

BUG: 128541828

Change-Id: Iee1dc8d4a429d759d9ab3f4b84dc1ad13b698cdb

Change theme setting to JSONObject format.

Add JSON_OBJECT_VALIDATOR to Settings and add unit tests.

Fixes: 129543539
Test: atest coretests
Change-Id: I4fa7a1357d442c1fa04a7eee815d5c3a48cc5115

Cancel Metric activations

Cancel Metric activations triggered by atom matchers

Bug: 128218061
Test: statsd_test
Test: statsd_localdrive
Change-Id: I90a705d74725c2aa04025e18e1fa77ec4fefc522

Don't lock the profile after setting work challenge.

In TrustManager user state defaults to "locked", so previously
the profile would become locked after work challenge is enabled.

Test: manual
Bug: 121256032
Change-Id: Ic0f5b075b85ca66c28d51830b47ed015c10da45c

Merge "carrier-config: define imsi_eap_identity_sequence_int key"

Expose input region monitoring to system components.

This is to allow SysUI to detect system-wide gestures.

Test: manual
Bug: 123748692
Change-Id: Ic1e2cd6afea219d0e20b29e7150c9a42b75c7a67

[Multi-user] Clean up user state stored in the system user directory
when user is removed.

For non system users, backup state is stored in both the user's own dir and the system dir.
When the user is removed, the user's own dir gets removed by the OS. This code change ensures
that the part of the user backup state which is in the system dir also gets removed.

Bug: 127650374

Test: atest -v CtsBackupHostTestCases:android.cts.backup.MultiUserBackupStateTest

Change-Id: I4ea252e8e6da608e36ec3ac335666923d88a8748

Pass app's data dir to ART

Inform ART about the location of the app's data directory when setting
up the process. This is part of an optimization that has ART cache
verification data into that directory.

Test: compiles, boots
Bug: 72131483
Change-Id: Ic80526b6ee383733eb5860e66f6c608109d838fb