Merge "Set notification channel to MTP notification." into oc-dev

Merge "Add more JSDoc about ErrnoException" into oc-dev

Set notification channel to MTP notification.

Bug: 38252594
Test: Manually connect MTP device to Android
Change-Id: Ib7ab12b2401d36dbb4cca2fd9c18cbd3505ed9ef

Merge "AudioEffect: return DEAD_OBJECT for FAILED_TRANSACTION" into oc-dev

Merge "Fix issue #38244875: Bring back testChargingConstraintFails" into oc-dev

Merge "Updating Overview to work with PiP" into oc-dev

Merge "Discover primary service by UUID for PTS tests (4/4)" into oc-dev

Merge "Use ArrayList as children" into oc-dev

Merge "Don't specify screen orientation for ReceiverActivity." into oc-dev

Merge "Specify behavior around DocumentsProvider#openDocument()." into oc-dev

Merge "ViewRootImpl: Guard against null in performMeasure" into oc-dev

Merge "Fixed leak of app token in exiting list." into oc-dev

Merge "Fix interaction of magnification and Final-crop." into oc-dev

Merge "Restrict access to instant app data in usage stats" into oc-dev

Merge "SurfaceView: Reset window stopped state when attaching to window." into oc-dev

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

Fix issue #38244875: Bring back testChargingConstraintFails

- Add new "get-job-state" shell command that the tests can use
  to find out what is going on with a job.
- Fix an issue in the battery controller where it would not
  immediately stop with a "battery not low" constraint when
  the battery becomes low.
- Also an optimization to rescheduling jobs, for every scheduled
  job that is not ready don't look to see if it is active.
  Instead, go through the active jobs and see if any are no
  longer ready.

Test: bit -t CtsJobSchedulerTestCases:*

Change-Id: I5611886653258ca337eee97c5ee1b9b3dfdb6d85

Specify behavior around DocumentsProvider#openDocument().

Add supported modes and contracts on receiving unsupported modes.
Test: Builds.
Bug: 11279339

Change-Id: I8c403f6168baadb354813d4c1e630bb8662ddb34

Fixed leak of app token in exiting list.

- Remove app token from exiting list when it's last window is removed.
- Remove app token from exiting list when it's removal isn't delayed.

Test: Launch a few apps and press back and make sure the exit app to
list dump is empty.
Fixes: 38337456

Change-Id: Ia2f22c6f1add5f867e25583ec38ffccc71217ee7

Use ArrayList as children

Such that traversal is O(n) instead of O(n²)

Test: WindowContainerTests
Bug: 32668632
Change-Id: Ie52026a76a0f65c71ff6bd35d13b3d0c5d44bb3f

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

Merge "Optimize hot launching recents" into oc-dev

AudioEffect: return DEAD_OBJECT for FAILED_TRANSACTION

Test: CTS AudioSecurityTest
Bug: 38346284
Change-Id: I96b4995c2d11b4a33c4b2fa6987922e26c91d336

Restrict access to instant app data in usage stats

- Events are obfuscated based on whether the app was instant or not at
the time each event was logged.

- UsageStats are obfuscated based on whether each app is instant or
not at the moment.

Bug 38202133
Test: Manual test using UsageStatsTest and instant apps

Change-Id: I3c74309196b88d010d317cb0dd6749bf4624e876

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

ViewRootImpl: Guard against null in performMeasure

In particular we are seeing this in the call sites from performTraversals
in monkey crashes. I don't have exact repro but it seems like a feasible
state to get in to...for example...WindowManagerGlobal#addView can trigger
removal of a dying view immediately without respect for the mIsInTraversal
flag when it calls doDie(). This means we can dispatch detached from window
setting mView == null while performing a traversal. There's some question
about why this doDie is even required but...seems a little nerve wrecking
to change at the moment and it seems best to just guard against null for now.

Test: Monkeys will test.
Bug: 37343098
Change-Id: I94f2569c1ef70819c083f2b2b34b59622e6c6260

Fix interaction of magnification and Final-crop.

In the case of magnification we simulate a transformation to screen space
by transforming all windows with the MagnificationSpec values. This means
we need to transform other values which are in "real screen-space" as well,
like final crop.

Bug: 36129852
Bug: 38322835
Test: Manual
Change-Id: I3215ffb75ba66a29135a79bcb23da51c828739fc

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

SurfaceView: Reset window stopped state when attaching to window.

We may be stopped, removed from the view hierarchy, and then only
attached again after the activity has been restarted, missing
our WindowStopped callback to set mWindowStopped=false. At this point
we are being added to a visible view, or in ViewRoot#performTraversals
so we can assume we are not stopped.

Test: Manual from bug.
Bug: 37682805
Change-Id: Idf8e061fb7f83b00992a274c7dd704f9e0fcff5f

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