Discover primary service by UUID for PTS tests (4/4)

Bug: 38123054
Test: manual
Change-Id: I89e088e60f4325f1ece59d22efda0c907a3b716a
(cherry picked from commit fe2bf16a2b287c3c748cd6fa7c14026becfe83ff)

Merge "Always add the instant app installer" into oc-dev

Merge "Fix broken retaildemo tests" into oc-dev

Merge "Check for mAttachInfo nullness when getting the autofillable drawable." into oc-dev

Merge "Optimize hot launching recents" into oc-dev

Always add the instant app installer

Even if we filter out an instant app because the caller doesn't
have access to it, always make sure to add the installer when
we have the possibility of an instant app.

NOTE: This does not have CTS tests since it's impossible to test
without involving the resolver. CTS tests will be added in MR1+
when we can make changes to the APIs and add the necessary methods
to mock out the resolver service.

Change-Id: I92c88eb04520c0c5e0c58db6d668ed0a0e552a35
Fixes: 38268052
Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.EphemeralTest

Merge "Remove deprecated recommendation request code." into oc-dev

Merge "catch USER_ALL in badging check" into oc-dev

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

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

Fix broken retaildemo tests

Bug: 38162695
Test: runtest frameworks-services -c
com.android.server.retaildemo.RetailDemoModeServiceTest &&
runtest frameworks-services -c
com.android.server.retaildemo.PreloadAppsInstallerTest

Change-Id: Iac60202414753b3e04e89cb29271cbd150a3acdd

Merge "ignore IllegalArgumentException during uninstall" into oc-dev

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

Check for mAttachInfo nullness when getting the autofillable drawable.

Test: manual verification on Caviar (automated test will be added later)
Test: CtsAutoFillServiceTestCases pass

Bug: 38341498
Fixes:  38323841

Change-Id: I15cc792de87987cc19a229c2ab2dfc317877f7ec

catch USER_ALL in badging check

Bug: 38264652
Test: runtest systemui-notification
Change-Id: Id482ed4984eef83ad0e9876acda5addb65aca5f9

Merge "Invoke BLKSECDISCARD to securely delete sensitive data" into oc-dev

Merge "Also add starting window when activity is not alive" into oc-dev

ignore IllegalArgumentException during uninstall

It's expected and normal to throw an IllegalArgumentException when
trying to uninstall a package that doesn't exist. Ignore it if it
gets thrown.

Bug: 38197362
Test: bit FrameworksCoreTests:android.content.pm.PackageManagerTests#testFlagFManifestI
Change-Id: Ie0d90010678b09c91747f6e5c4655691951ad3a9

Merge "Remove internal listener when cloning" into oc-dev

Optimize hot launching recents

Rearrange how we generate the transition specs, which involves
creating a thumbnail on the mainthread (about 10ms on large
devices): First, we put launching the activity onto a handler
thread (with default priority), to free up the main thread. Then,
we immediately start generating the thumbnail such that when the
future calls us we have the generated spec already handy.

For that we need to be able to supply a specs future into
ActivityOptions, to avoid race conditions. Furthermore we need to
make sure not to call into WM while creating specs, to avoid WM
lock contention.

Test: App -> Recents -> Same app, inspect app transition logs
Test: Double tap recents for quick switching

Bug: 32668632
Change-Id: I6001e29145f8e56deb9c4ead46c53c87c9191436
Merged-In: Ic6ec65c2560f67cade3b5ddde9f79ee13e9ba32c

Merge "Fix spinning TaskPersister" into oc-dev

Merge "Immediately report drawing" into oc-dev

Merge "Fix back stack problems due to postponed transitions" into oc-dev

Merge "Fixes crash (race cond) when using toast that is immediately cancelled" into oc-dev

Also add starting window when activity is not alive

The fact whether the process is running or not is not necessarily
a reason to not show a starting window. Sometimes the process with
an activity gets killed, but later gets restarted because of some
broadcast or service without recreating the activity. In this
case, we still need a splash screen to hide the recreation delay,
which is usually as expensive as if the process is not running.

Test: Open Calendar, kill `pid calendar`, reopen it, make sure
starting window is shown.
Test: As above but with a couple of other apps - with and widhout
trampoline activities.
Test: Boot freshly and open a couple of apps from recents

Change-Id: I8c4f928fca77b5446cab55c89bc69adbaaaa8da3
Fixes: 37951698

Immediately report drawing

No need to wait on the next relayout - this can only delay the
transition. Makes hot launches a lot more consistent.

However, this made it too fast! We then hit a race condition when
the app transition was already starting but no other layout was
done yet. When another layout was executed we noticed that we need
to report resized for the starting window, clearing it's drawn
state, which set startingDisplayed=false, which jumped the app
window animation to the end.

To fix this, make sure not to report another resized immediately
after the initial layout, as the client already knows the latest
(because it calls relayout at some point before it starts drawing).

Also fix "animating" async systrace for better analysis.

Test: Open/close size-mismatching task snapshot 100 times, ensure
no animation skipped.
Test: Look at app transition logs, ensure more consistent.
Test: Overall system sanity testing (open a couple of apps/dialogs
etc).

Bug: 32668632
Change-Id: Id795cd6a84f22e6a619089cb9554fc5033477ad2

Invoke BLKSECDISCARD to securely delete sensitive data

Bug: 34600579
Test: manual - change device lock under synthetic password, verify
      old data on disk is erased.

Change-Id: I247bd1f095dd27335e671981f9e2d77e149af84f
Merged-In: I247bd1f095dd27335e671981f9e2d77e149af84f

Fix spinning TaskPersister

Wait on the lock if the queue is paused instead of
just spin-looping.

Test: Close app, make sure screenshot gets persisted.
Bug: 36631902
Change-Id: Id7940468391d6cdfc74bb9341c1639f72d469387

Fix broken DUN evaluation logic

Test: as follows
    - built
    - booted
    - flashed
    - runtest frameworks-net passes
Bug: 32163131
Bug: 38152109
Bug: 38186915
Merged-In: Ib9a469b16a598c47b615a15fc7cce420bfe4b8b7
Merged-In: I293b4cd57dcd1968c2b38f4e92e24cc0d2e135a9
Change-Id: Id4803504e825290d62c0b51a2b9bd664dcef30b1
(cherry picked from commit 54f2f3704e4ccf2c70ff61aab4e7ea9d59918e46)

Import translations. DO NOT MERGE

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

Fixes crash (race cond) when using toast that is immediately cancelled

Running cancel after toast is shown and adding some UI stress (or sleep
on UI thread) causes a crash from toast when trying to add the toast
window to the display. The toast must be triggered from app that is
above N MR1 (25).

The steps that crash the app are:
1. Show toast (Toast.makeText(...).show()), window token is created
2. Immediately cancel toast (Toast.cancel()), window token is removed
3. Stall UI thread (Thread.sleep, heavy task), both show and cancel
events are queued to UI thread from window manager
4. Crash trying to add toast but no window token exists

In Toast:handleShow(), the mNextView is required to add the toast to
display, if the mNextView is null before posting to window manager, then
when handleShow() runs later, it will ignore adding the toast to
display. The issue before is that mNextView is set to null after cancel
runs back from window manager in UI thread but the show post will always
happen first. Therefore set mNextView to null at the beginning of
cancel will ignore adding the toast to display and avoid the crash.

Bug: 37606432

Test: manual - write an app to Toast.show(), Toast.cancel(), then
Thread.sleep(), set app's sdk usage above 25 (N MR1) and show the
toast

Change-Id: I352e296c47b1b8776c78b6b0943b1dc809963026

Merge "MediaBrowser: Allow to connect while disconnecting" into oc-dev

Import translations. DO NOT MERGE

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

Merge "Revert "Revert "O is API 26""" into oc-dev

Merge "Revert "Revert "Resource ID fixup.""" into oc-dev

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

Revert "Tweak test app makefile with extra resources"

This reverts commit f0435e183906dc39432fb4c937854d3737de3adc.

Bug: 30240339
Change-Id: I6cc03188d5666eded5869c583db76e02b6f90794

Merge "Update TimePicker mode switch icon." into oc-dev

Import translations. DO NOT MERGE

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

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

Update TimePicker mode switch icon.

Previously the icon was an event icon, but a clock icon is
more appropriate so we are switching to that instead.

Bug: 37351390
Test: Open time picker
Change-Id: I47e6caf3c341c10264168004628288fd00e4601a

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

Fix back stack problems due to postponed transitions

Bug 38277003

The back stack was being moved while executing operations and
then again when the postponed transaction was executed. It should
only be done once.

Test: Ie47e1f2f158325b9cfd6edb5c40c65d764ff9056
Support Lib Change: I3159c2345a7b77fa82f1c602f4639f51b5a47980

Change-Id: I3b7a032e7e8a9aec565157d42dcaa7232b256ae8

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

MediaBrowser: Allow to connect while disconnecting

Bug: 37241378
Test: manual, passed MediaBrowserTest and MediaBrowserServiceTest
Change-Id: I8428c4fcb9855f1a4d898419d7f1bb09495eb8d9

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

Remove internal listener when cloning

BUG: 38270215
Test:  adb shell am instrument -w
com.android.frameworks.coretests/android.support.test.runner.AndroidJUnitRunner

Change-Id: Ia80d5f8958e320ad7d51ef7a08168763c8988146

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