Merge "Support recursive static shared lib dependency" into oc-dev

Merge "Preserve non-floating state when entering pinned stack." into oc-dev

Merge "Tweak test app makefile with extra resources" into oc-dev

Merge "Require targetSdkVersion>=O for updated graphics driver packages" into oc-dev

Merge "Changed alert window notifications to use single channel" into oc-dev

Support recursive static shared lib dependency

Test: Updated the static shared lib CTS tests to verify that
      All static shared lib CTS tests pass

bug:35359862

Change-Id: Id4a0dabbc9d8e98962e92b6f537f6ffffa63b585

Require targetSdkVersion>=O for updated graphics driver packages

In O, graphics drivers are loaded into a new restricted linker
namespace. Drivers built for previous versions of the OS may not work
under those restrictions, so require an updated driver package to
declare compatibility by setting targetSdkVersion >= O.

Bug: 34228255
Test: manually construct packages with and without
      targetSdkVersion >= O, confirm driver is used/not-used as
      expected.
Change-Id: I4518360433a6de5c6e1e792a6eedddf8c6bf4394

Merge "Don't request account access for apps if it is explictly made not visible to them." into oc-dev

Merge "Post setSwitchingUser to main thread" into oc-dev

Merge "Keyguard shouldn't listen to FP if bouncer showing and keyguard dismissing" into oc-dev

Merge "Center switch vertically with primary text" into oc-dev

Changed alert window notifications to use single channel

Per UX request.

Change-Id: Iefe1be52743519af5ee1481b59613c87f16bdf7d
Fixes: 37422870
Test: Verify alert window notification channel.

Merge "Fix drm hal resource leaks" into oc-dev

Merge "Ensure that we check against the pending state while suspended." into oc-dev

Keyguard shouldn't listen to FP if bouncer showing and keyguard dismissing

There's a race condition if user unlocks from bouncer and an app underneath
uses FP does onResume()

If a user has an application that's authenticating via FP underneath
keyguard and the user dismisses keyguard via bouncer, the app underneath
will get kicked out since onKeyguardVisibilityChanged(false) calls
shouldListenForFingerprint() during this race.

Keyguard shouldn't listen to FP if bouncer is showing but keyguard is
dismissing

Fixes: 37967985
Test: 1) open FP settings
2) enter keyguard and go to bouncer
3) unlock keyguard via pin/pattern/password (NON FP method)
4) tap any finger on sensor (should vibrate)

Merged-In: Ie38d55e6a1ec9b49f9df0c7520d0bc451315c161
Change-Id: Ie38d55e6a1ec9b49f9df0c7520d0bc451315c161

Merge "Map isolated processes to correct uid" into oc-dev

Merge "Fix Shortcut crash due to invalid Icon type (e.g., all adaptive bitmap shortcuts)" into oc-dev

Fix Shortcut crash due to invalid Icon type (e.g., all adaptive bitmap shortcuts)

b/38304824
Test: cts tests in ag/2262246 and also full build and tried the system image.

Change-Id: I8f021aa2bb1a0e99597bc00064ab1e9d85569bcb

Post setSwitchingUser to main thread

touching view hierarchy should only occur on UI thread

Bug: 37167272
Test: keyguard still works, used Log to see that switching
      still gets called
Change-Id: I45ed9894a8f08c38da40de44f84e99cddb683014

Merge "Add null check to a11y call." into oc-dev

Merge "Use placeholderCount-1 if using default layout." into oc-dev

Fix drm hal resource leaks

bug:36408047
test: manual testing to verify leaks are fixed
Change-Id: Ic6a29572874a9fe30f36e7fdabb68d2ddeffb278

Merge "Fix a deadlock in LockSettingsService" into oc-dev

Center switch vertically with primary text

Fixes: 38248336
Test: manual
Change-Id: I48bfb244abff47c16ca1f24c0027448a2a238015

Merge "Fix number of channels in inline notification settings" into oc-dev

Fix broken MenuItem coretests

- Fix MenuItemImpl setShortcut bug caused when method signature was
changed after API review
- Remove outdated MenuItem coretests and move others to CTS

Bug: 38114634
Test: Run `cts-tradefed run cts-dev -m CtsViewTestCases -t
android.view.cts.MenuTest`

Change-Id: Iebb7e314cbb3f812fcfeb3f95797f1cf1bcfbae2
(cherry picked from commit d70d2e6efccf186af08289eb5dc5fe1835b753ed)

Fix a deadlock in LockSettingsService

Order of taking locks in LSS: mSeparateChallengeLock -> mSpManager -> this.
Since getGateKeeperService().getSecureUserId() will end up calling
getSeparateProfileChallengeEnabled() which tries to acquire mSeparateChallengeLock
, we shouldn't be holding this lock while calling getSecureUserId()
Also spell out the order of taking locks in a comment.

Test: while device lockscreen is being changed, run "adb shell dumpsys lock_settings"
      and verify device is not locked up.
Bug: 37663848

Change-Id: I6b780ab5d1e33b07392f0ab0df29ecc3956f89ff

Merge "Hadler USER_ALL when uninstalling static shared lib" into oc-dev

Merge "Query for static shared libs as well when needed" into oc-dev

Hadler USER_ALL when uninstalling static shared lib

Test: manual - cannot uninstall an lib with cross user dependency
       All static shared lib CTS tests pass

bug:36632264

Change-Id: Idd11405b27ccd8cacc2a50814e1317356689b7d1

Merge "Relayout before creating surface" into oc-dev

Relayout before creating surface

Since we create starting window surfaces immediately (instead of
waiting for app visibility), we hit a weird corner case in which
the surface was created before the window was laid out. Thus, a
little bit later, we had to call SurfaceControl.setSize, which
made the whole transaction blocking, meaning that we blocked until
the next vsync before returning to the client (starting window).

Fix this by running a layout if the window wasn't laid out yet.

Test: Open hot app, see latency go down by an average of 8ms (up
to 16ms better!)

Bug: 32668632
Change-Id: I9817afc72127e32d7f8f4e497e0dccd7f180a9c0

Query for static shared libs as well when needed

System components can see static shared libs but have to
explicitly request that. This change explicitly requests
static shared libs as well in the package manager shell
command, the package installer and the package manager
where needed.

Test: Static shared lib CTS tests pass

bug:36632271

Change-Id: I26804571be327888b830e2f6bccdcbaa820f763c

Merge "Clear calling identity before binding instant app resolver" into oc-dev

Merge "CaptivePortal: login activity UI improvements" into oc-dev

Merge "Fix MediaSessionManager.Callback to be called with the media button session" into oc-dev

Merge "Launch voice assistant by the KEYCODE_MEDIA_PLAY_PAUSE" into oc-dev

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

CaptivePortal: login activity UI improvements

Similarly to commit 2e0915f14384901f25a41d698b39ef0add201550 for the
carrier portal login, this patch changes the default settings of the
webview used for the system captive portal login activity to allow
better user experience on "wide" login pages designed for desktop.

Test: manually tested with captive portals.
Bug: 31813936, 19228946, 36532213
Change-Id: Ib84fd351e47e951d24f297bc7de1b035b51cf24f
Merged-In: I2579994da37f3b0f4c08e24e59c81f31835ab832

(cherry picked from commit f7eea406c950e447608c6f62de05158334682365)

Merge "Fixed and improved (for reuse) PendingRequest lifecycle." into oc-dev

Start a new session on manual request after session is "gone".

When the autofill service returns a null FillResponse, the session is marked
"gone" because the service cannot autofill it. But there might be cases where
the view structure change and it's now autofillable, so need to allow users
to manually request autofill again in such cases.

Fixes: 38205945
Test: CtsAutoFillServiceTestCases pass
Test: LoginActivityTest.testAutofillManuallyAfterServiceReturnedNoDatasets()
Test: LoginActivityTest.testAutofillManuallyAndSaveAfterServiceReturnedNoDatasets()

Change-Id: I9b23c255e563dd0646bf266d31ddb10dcc4f7f6d

Merge "Don't print sensitive data on AssistStructure.dump()" into oc-dev

Don't print sensitive data on AssistStructure.dump()

Test: manual verification

Bug: 38205945
Change-Id: Ic184ee6da313a2c0f8e0dd11979f6985d8023140

Merge "Fix issue #38210653: Few more tweaks of job scheduler" into oc-dev

Merge "Fix the status icon colors in QS" into oc-dev

Merge changes I7b5504d0,I10c263dc into oc-dev

* changes:
  Get rid of a lot of binder calls
  Fix recents entry delay

Import translations. DO NOT MERGE

Change-Id: I4c22a6f78ff012791677fe6935351f008108dcb9
Auto-generated-cl: translation import

Get rid of a lot of binder calls

Lead to delays when entering recents.

Test: Open recents
Bug: 32668632
Change-Id: I7b5504d0c11208d9c4d55de2eace6407d186f25c

Fix recents entry delay

- Start loading things after we drew our first frame.
- Don't load high res task until low res tasks have been loaded:
In case the snapshot is already in cache in system_server the
low-res task loader will load the full res snapshot, leading to
GC pressure due to registerNativeAlloc. Now, if both task loader
load the same snapshot at the same time we are very likely
to trigger a blocking GC alloc.

Test: Open recents, look at systrace
Change-Id: I10c263dc8929742611ad9fbb32f65bc8ac3100bd
Fixes: 36851903
Bug: 32668632

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

Merge "Couple autofill fixes." into oc-dev

Merge "Allow users to quickly focus the minute field in TimePicker." into oc-dev

Merge "Fix recent apps in system UI for Skia pipeline" into oc-dev

Merge "Introduce result ranges for activity start return results." into oc-dev

Merge "OMS: Setup state for users on boot and when added" into oc-dev

Merge "Add new reason code for IMS Call ended during conference merge process" into oc-dev

Merge "Revert "Respect PopupWindow.setOverlapAnchor when above anchor"" into oc-dev

Map isolated processes to correct uid

This was missed in a recent addition to track cpu times at freq.

Bug: 34133340
Test: manual
Change-Id: Ie3720a76a1201c787601bb7419cb9eba47b2aa27

Fix issue #38210653: Few more tweaks of job scheduler

- Now keep track of the time a job was enqueued, and order
  the pending list by that.
- Added configuration constants for rescheduling: maximum
  times to reschedule, minimum backoff times.
- Fixed printing of active jobs -- the method to get the current
  JobStatus was old and didn't require the caller to hold a
  lock, so made a copy, which didn't contain all the data we were
  interested in.  Now with our simple locking, we can just make
  that require the caller hold a lock and return the real
  JobStatus object.
- Include oom_adj and procstate when printing information about
  processes being killed.
- Expanded documentation of BroadcastReceiver.goAsync().

Test: bit CtsJobSchedulerTestCases:*

Change-Id: I2e45f181e45be9836c74cbff1b844ffdf6e93019

Merge "Add missing break statements in FingerprintManager" into oc-dev

Merge "Updating expand icon asset." into oc-dev

Merge "Fix broken PackageParserTest" into oc-dev

Merge "AAPT2: Allow <uses-permission-sdk-23> in manifest" into oc-dev

Merge "TextView: Invalidate selection action mode on "select all"." into oc-dev

Merge "Fixed an issue where the media notification wouldn't have contrast" into oc-dev

Fix recent apps in system UI for Skia pipeline

Enable HW Bitmaps for Skia pipeline just enough to make
recent apps list working by adding support for BitmapShader.
Drawing HW bitmaps in a canvas is also supported.

Test: recent apps work, HWUI unit tests pass, CTS tests pass.
bug: 38136140
Change-Id: Ibd06c859c86dc213310d5ce5272497e1882d0cc6
Merged-In: Ibd06c859c86dc213310d5ce5272497e1882d0cc6

Merge "strong fp lockout after 20 failed attempts" into oc-dev

Merge "Do not perform backup/restore of instant apps" into oc-dev

TextView: Invalidate selection action mode on "select all".

Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
bit CtsWidgetTestCases:android.widget.cts.TextViewTest
Change-Id: I0b5ebb8d6f1af1a9938151f758a2feedb14fcb9f
Fixes: 38244876

Import translations. DO NOT MERGE

Change-Id: Ideae955ce1ddea3d17038235c4ed55127d7b7541
Auto-generated-cl: translation import

Merge "Add command to retrieve the instant app resolver" into oc-dev

Merge "Properly invalidate text classification cache." into oc-dev

Merge "Added redundant focusable attribute read in TextView" into oc-dev

Introduce result ranges for activity start return results.

The return result from starting activity actually represents two
pieces of information. First, it conveys whether the activity
started. Secondly, it conveys whether there was a fatal error if
the activity did not start. Many parts of the code assume that a
value greater than or equal to the defined success means that the
activity successfully started. This is not the case as there are a
number of results greater than success where the activity does not
start.

This change addresses the issue by introducing three distinct
result ranges. The first represents results where the activity
could not start due to a fatal error. The second represents results
where the activity did not start due to a non-fatal error. The last
range represents successful activity starts. Two convenience methods
have been added to ActivityManager to return whether the result
represents a fatal error and whether the result was successful.

Change-Id: Ifaf844c353641a28b03b3c2d7b6be053fd9b8b44
Fixes: 38021882
Test: cts-tradefed run cts-dev --module DevicePolicyManager --test com.android.cts.devicepolicy.DeviceOwnerTest#testLockTask_deviceOwnerUser

Merge "Update ShortcutManager/Service javadoc/impl around sizing" into oc-dev

Merge "TextClassifier: Ensure url scheme is in lowercase" into oc-dev

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

Merge "Work on issue #38242094: Activity manager oom adj computation seems broken" into oc-dev

OMS: Setup state for users on boot and when added

Some users never get switched to (managed profile/work profile) so the
overlay state for a user would never be setup (but they could still show
UI and apps).

This change ensures that user state is setup after an OTA to android O,
and whenever a user is added.

Bug: 37899201
Test: manual (add user via Device Admin sample: vendor/google/tools/DeviceAdminSample)
Change-Id: If214e26e39b18c2861794baf5c608a47d536e5ff

Ensure that we check against the pending state while suspended.

Bug: 38192462
Test: Launch PIP, try to make PIP fullscreen.

Change-Id: I575f57492506224b8ee9f247f3c5903c812a04d6

Couple autofill fixes.

- Fixed PFLAG3_NOTIFY_AUTOFILL_ENTER_ON_LAYOUT value - it was overlapping with
PFLAG3_ASSIST_BLOCKED.
- Added missing permission on methods called by Shell command.
- Improved debugging.

Bug: 38173625

Test: android.autofillservice.cts.MultipleFragmentLoginTest#loginOnTwoFragments

Change-Id: Ie7880c97d38298e06134709c5c2e9e38fa1bacbb

Adding AOSP copyrights header

Test: N/A

Change-Id: Iabec646a1f8087480f899c5b4996122a9f4bd13a

Fixed and improved (for reuse) PendingRequest lifecycle.

Fixes: 38173625
Test: android.autofillservice.cts.MultipleFragmentLoginTest#loginOnTwoFragments

Change-Id: Ic9ba3cb51048296384d42670cc71db78a0e8fcf6

AAPT2: Allow <uses-permission-sdk-23> in manifest

Bug: 38269610
Test: build tools/aapt2/integration-tests/AppOne
Change-Id: I1d0b87aa36f6bf82aab146339acf8848166b218d

Merge "Updating AppWidgetServiceImplTest as per Mockito changes" into oc-dev

Updating expand icon asset.

Bug: 38237934
Test: Launch activity in PIP, tap to show expand icon
Change-Id: Icae85ff27accd1e2143836541587e3bedc17570b

Add missing break statements in FingerprintManager

Fixes: 38268065

Test: manual

Change-Id: I64ac92b2c5682bcaa494f43ff5defdf9915c5a65

Merge "Batterystats BLE results counter takes in count" into oc-dev

Merge "Fixed exceptions during test tearDown" into oc-dev

Merge "Setting Session commited broadcast when an existing apk is enabled for a different user" into oc-dev

Do not perform backup/restore of instant apps

This keeps the install flow fast, and also happens to mean that
there is feature parity in this regard between pre-O and O+
instant app handling.

Fix #38263351
Test: manual

Change-Id: I8eb08c12d800f5288583e792621920c9876efdfc

Merge "SystemUI: add NETWORK_SETTINGS permission" into oc-dev

Import translations. DO NOT MERGE

Auto-generated-cl: translation import

Bug: 38120923
Change-Id: I3ebe901771f444c2986b15a52f4d16bd342fac37

Merge "Merge "Add PID info into ANR crash log header" am: 2fe972c822 am: 5d57502c89 am: 5dbe26f0f1 am: 4ae323995d" into oc-dev

Merge "Fix JobSet getJobsByUser" into oc-dev

Work on issue #38242094: Activity manager oom adj computation seems broken

This shouldn't change any behavior, but improve the reason we
report for each process's oom_adj / proc state level.  There are
two significant things going on here:

1. We now consider a bump up in proc state to be just as significant
as a change in oom_adj, and accordingly update the reason when this
happens.  Given how many proc stats we now have mapped to some of
the single oom_adj levels, this matches your expectation to have the
reason be why it is at that proc state and not just some random
other thing at that level.

2. There is special handling of an app at the top state connecting
to another app, deciding the actual state to apply at the end.  This
was not at that point updating the reason, so anything the top app
is connected to would get the top state but myseriously have some other
reason, looking very broken.  We now propagate the reason over.

Test: manual

Change-Id: Idecbe206d73e7c4cbd989ef6faf3b1679e06c088

Allow users to quickly focus the minute field in TimePicker.

Set imeOptions to be actionNext for the hour field of TimePicker to
allow users to jump to the next field using the keyboard.

Bug: 36863243
Test: Manual, keyboard now shows next button instead of enter that moves
      focus to the next field.

Change-Id: Ia9d6137d14579ba5b99fbb714a78423cc75b5fa0

Merge "Set notification TAG and style in TV PIP notification" into oc-dev

Fix the status icon colors in QS

Requires lots of theming magic

Test: visual
Change-Id: I1cd28b570f30989b496065c0e06a5aca461deb41
Fixes: 37619734

Merge "Avoid activity leak via Companion callback" into oc-dev