Merge "Add hidden API startActivityForResultAsUser" into oc-dev

Merge "ViewRoot: More surgically fix child-life time." into oc-dev

Merge "Binder: Be forceful about a forceful exit." into oc-dev

Merge "Remove supportsDismissingSelfWindow attribute of IME" into oc-dev

Merge "Stack APPLICATION_MEDIA_OVERLAY windows with relative layering." into oc-dev

Merge "Minor text fixes." into oc-dev

Merge "AAPT2: Generate R.txt" into oc-dev

Merge "Catch exceptions in Vibrator to maintain compatibility" into oc-dev

Merge "OMS: Fix bug where overlay settings wouldn't persist" into oc-dev

Merge changes from topic 'api-review' into oc-dev

* changes:
  MediaCas: combine openSession methods
  MediaCas: address comments for API reviews

Stack APPLICATION_MEDIA_OVERLAY windows with relative layering.

See accompanying frameworks/native commit
 "SurfaceFlinger: Add parent-less relative layering" for a full explanation.

Test: Manual of bug repro steps. Plus tests for new SurfaceControl functionality included in frameworks/native.
Bug: 36693738
Change-Id: Ic54598117c1f44a206d33f03d0cc463fbef43fcc

Merge "Use Context#startForegroundService in MTP provider" into oc-dev

Merge "Report app data fixup and prepare timings" into oc-dev

Merge "Revert "Make scrollbarSize in Material Design theme configurable"" into oc-dev

Merge "Add set-mode command to turn battery saver mode on and off." into oc-dev

Merge "Update NPMS internal state when app idle parole state is changed." into oc-dev

Merge "Fix regression when removing task." into oc-dev

Merge "Work on issue #36427964: SecurityException: Unable to find app..." into oc-dev

Merge "Added strings for thermal shutdown warning" into oc-dev

Merge "Dump controller for AppWindowToken" into oc-dev

Merge "Revert "Ensure visible activities when device is unlocked."" into oc-dev

Revert "Ensure visible activities when device is unlocked."

Causes issues with window life-cycle...

Bug: 37119770
Bug: 37244415
Bug: 37281544
This reverts commit 7004a8801d0a60013681a86043bab74b4d42d680.

Change-Id: I0039db6e44488cd82ef745aa979bc8549fd8aeed

Merge "Expose more settings to Instant Apps" into oc-dev

Merge "Fix focus ordering with duplicate ids" into oc-dev

Merge "Revert "Make selection end handle stick to selection at line end."" into oc-dev

Merge "Fix auto-pip visibility issues." into oc-dev

Merge changes from topics 'KeyboardFocusHighlight-oc-dev', 'KeyboardFocusHighlight_ mDefaultFocusHighlightEnabled-oc-dev', 'KeyboardFocusHighlight_ hasFocusStateSpecified-oc-dev' into oc-dev

* changes:
  Fix broken CTS tests.
  Draw a default focus highlight if needed.
  Add a public API defaultFocusHighlightEnabled.
  Detect unhandled keyboard focused state.

Add set-mode command to turn battery saver mode on and off.

Affected test will be modified in ag/2101279

Test: adb shell cmd battery unplug && adb shell cmd power set-mode 0
Bug: 31944272
Change-Id: Ia88e7e164aa9b8b4d0ab51f607a64b35e2159273

Merge "clear calling identity before calling settings provider" into oc-dev

Merge "Add p2 option in openLogicalChannel." into oc-dev

Merge "Allow appops command to modify uid ops." into oc-dev

Update NPMS internal state when app idle parole state is changed.

Fixes: 37250313
Test: 1. cts-tradefed run singleCommand cts-dev --module CtsHostsideNetworkTests -t \
      com.android.cts.net.HostsideRestrictBackgroundNetworkTests
      2. runtest -c com.android.server.NetworkPolicyManagerServiceTest frameworks-services
Change-Id: I5fcac397116a07fe9015b7dabf5bca57b0b23176

Merge "[DO NOT MERGE] Allow multiple toasts for the focused app" into cw-f-dev am: 39c3d2cf16
am: 91ed4d7553  -s ours

Change-Id: Ia744f56107b83b6a5d5e236d24641bd7db71ab02

DO NOT MERGE: Remove the development privelege level from SET_DISPLAY_OFFSET. am: 0a8ecc6886
am: f72dd4142b  -s ours

Change-Id: Ia99beb815b720219daecfdc9446c5ff2c4d1b16f

Merge "Add autofill feature API" into oc-dev

Merge "[DO NOT MERGE] Allow multiple toasts for the focused app" into cw-f-dev
am: 39c3d2cf16

Change-Id: I3ddf7e0ceadd7c179f3c7de198658af190985f1b

Merge "Add API to get a uid's current importance." into oc-dev

DO NOT MERGE: Remove the development privelege level from SET_DISPLAY_OFFSET.
am: 0a8ecc6886

Change-Id: I262e7497e9256f708847ba7c767f202c91aa1739

Merge ""bmgr restore" really should wait until operation finishes." into oc-dev

Merge "Fix issue #37220301: Allow broadcasts with permissions to not be restricted" into oc-dev

Merge "Deprecate findAddress() method" into oc-dev

Merge "[DO NOT MERGE] Allow multiple toasts for the focused app" into cw-f-dev

OMS: Fix bug where overlay settings wouldn't persist

The dead user check was using a List.removeAll() call to find
the subset of users that have been removed and need deleting
from the overlay settings. The target class, UserInfo, does not
implement equals(), therefore no UserInfos would be removed from
the set and all user IDs would be candidates for deleting.

This change avoids the UserInfo class entirely and manipulates userIds.

Bug: 36099320
Test: Pick a theme and reboot.
Test: Theme should still be applied on next boot.
Change-Id: I1ee57839515460bf578664dfe1bd67df7d10d041

Merge "Make QS light" into oc-dev

Dump controller for AppWindowToken

To help with debugging.

Bug: 36097078
Test: N/A
Change-Id: I72108b918d839b67efe8b8eed65a33775bd94dbc

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

Merge "Allow instant app resolution for any user" into oc-dev

Merge "Update Typeface.Builder API to reflect API feedback." into oc-dev

Merge "Rename WEB_ACTION_ENABLED to INSTANT_APPS_ENABLED" into oc-dev

Merge "Add documentation for 'am start-foreground-service'" into oc-dev

Merge "Clear calling identity before verifying listeners" into oc-dev

ViewRoot: More surgically fix child-life time.

A more targeted version of "ViewRootImpl: Fix child lifetime".
In this version we avoid hijacking the window visibility callback
and introduce a new path.
Our contract has been that at any time after returning from
onStop, the WindowManager may destroy the clients Surface. We
see in setWindowStopped, we set the stopped state on the hardware
renderer in order to prevent further use of the Surface. However
there is nothing synchronously dispatching this change to child views
and so SurfaceView is not necessarily informed to release it's Surface
in time. Typically SurfaceView will be informed through the
onWindowVisibilityChanged callback. The signal producing this
is emitted from SystemServer prior to onStop but has to pass
through the clients handler thread. It seems this has always been
broken, and we have always had intermittent reports of EGL_BAD_ALLOC
scenarios. I speculate that elevation of WindowManager scheduling during
app close scenarios is perhaps making this more severe, as it seems to
ensure the WindowManger will win the race to destroy the Surface before
the client handler thread process onWindowVisibilityChanged (unless
the FIFO timeout is surpassed).

Test: Put Chrome in PiP. Turn screen off. No Crash!
Bug: 36561071
Change-Id: I9233ba8151c7f577b1d1044afc0c2ac3a65be4b4

DO NOT MERGE: Remove the development privelege level from SET_DISPLAY_OFFSET.

Bug: 36515029
Test: the build still works.
Change-Id: I39ae2c55b6b2b0d6547f75f4ef06e62e3e5f0b84
(cherry picked from commit 8e9ea907448c49f573dd045707e37dd14efdf152)

Merge "Improved multiuser support for A11y button/Magnification" into oc-dev

Merge "Fill UI polish" into oc-dev

Added strings for thermal shutdown warning

Strings for thermal shutdown warning were missing
so this CL adds them and a proto message value.

Test: SysUI tests still pass
Bug: 30994946
Change-Id: Ifd0b26248c2ebae5bcf32ecbea2566c14be7dc32

Merge "Updates reportChooserSelection to be synchronized with queryUsageStats in UsageStatsService, to fix a flaky test." into oc-dev

Merge "Don't offer snooze for foreground service notifications" into oc-dev

Merge "Disallow direct-boot aware activities to bypass work lock" into oc-dev

Fix broken CTS tests.

The default focus highlight should be necessary when the view is
attached to window.

Test: cts-tradefed run singleCommand cts --skip-device-info
--skip-preconditions --abi armeabi-v7a -m CtsViewTestCases -t
android.view.cts.ViewTest
Test: cts-tradefed run singleCommand cts --skip-device-info
--skip-preconditions --abi armeabi-v7a -m CtsWidgetTestCases -t
android.widget.cts.MediaControllerTest
Bug: 37082416

Change-Id: I5d6b0c00b5ba03c8f62e4f71be9336823a73134b

Draw a default focus highlight if needed.

This commit draws a default highlight for a focused
View if it's detected to have no state_focused defined
and its useDefaultFocusHighlight attribute is true.

When we detect a default highlight is needed, we show it
on top of the view to. Once we detect that it's no longer
needed, we remove it.

Test: Check that views without a focused_state in its
state spec have a default highlight when they get focused.
Bug: 35096940

Change-Id: Ifbe4bb9e1297d98845314e24d8b758f14e5987a9

Add a public API defaultFocusHighlightEnabled.

This commit adds a public API to turn on/off default
focus highlights. When a focused View is detected to
have no state_focused defined, we can use this API to
decide wether we should show a default highlight for it.

Test: cts-tradefed run singleCommand cts --skip-device-info
--skip-preconditions --abi armeabi-v7a -m CtsViewTestCases
-t android.view.cts.View_DefaultFocusHighlightTest
Bug: 35096940

Change-Id: I76b45d6bf5761641a0ed7f4d0b04cb325ed72b52

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

Revert "Make scrollbarSize in Material Design theme configurable"

This reverts commit 98329f4da372e2ab00565b88df785620856cfb65.

Bug: 35432058
Test: android.view.cts.ViewTest passes
Change-Id: I4c76450a47b581a8237108180806845f5f66ac39

Merge "Do not enforce paragraph constraint for copy ctr" into oc-dev

"bmgr restore" really should wait until operation finishes.

Test: manual test
Bug 37246838

Change-Id: Ice381dc250e2d2a59cff48152dd3c8d6897d0804

Merge "Add new "work queue" feature to JobScheduler." into oc-dev

Add API to get a uid's current importance.

This is kind-of useful to go along with monitoring
uid importance changes.

Test: none yet

Change-Id: Ic0f8418955d17ea21d06f49dcd9641bc5f32387b

Merge "Revert "Transition from Activity to Broadcast for TV Search"" into oc-dev

Merge "Change behavior of setBitmap to cleanly reset the canvas" into oc-dev

Merge "Remove unused public APIs." into oc-dev

Merge "Fix foreground importance bug in LocationManagerService." into oc-dev

Expose more settings to Instant Apps

Test: Settings are accessible from an instant app
Change-Id: Ib6e5f603184bc94c8a47f010e1609b3021765403

Merge "TIF: add review rating columns to programs" into oc-dev

Merge "Disallow task snapshot starting window for intent != ACTION_MAIN" into oc-dev

Merge "Fix NPE in TaskSnapshotController" into oc-dev

Merge "Updates for QS DND tile" into oc-dev

Merge "Visual changes to dual target tiles" into oc-dev

Merge "Ensure bounds are updated together." into oc-dev

Merge "Introduce FILE_ID for TTC fonts and variation fonts." into oc-dev

Add autofill feature API

We now have a software feature for autofill which can be used
by partners to disable it on low-end devices or form factors
for which autofill doesn't make sense.

bug:35956220

Test: manual (requires a custom build)

Change-Id: I6c06462ed9ca3ae93331700dce38a8c08dfd0722

Fill UI polish

1. Avoid jank when the suggesting list scrolled

2. Remove hard-coded values from max popup height

3. UX requested tweaks

4. Remove dead code

Test: manual

bug:37245055
bug:37179467

Change-Id: I6a6760edb06230e3d4925e23863667cfd3ac0601

Fix auto-pip visibility issues.

In the case that we defer stop for auto-enter picture
in picture, we should also defer changing the visibility. This causes
particularly awkward issues with SurfaceView as the views are left
in a severed state.

Test: Auto-pip Chrome from youtube. Things look alright!
Bug: 36355266
Bug: 36640131
Change-Id: I77de1c3eb9c61b03740cbe49f88dec1af2ed6577

Merge "Use the StorageStatsManager in FileCollector." into oc-dev

Disallow direct-boot aware activities to bypass work lock

Effectively reverting 89927b3cd96472c478a988d6c731cd09d412a043, which
allowed direct-boot aware activities in the work profile to show before
the profile was unlocked. This causes problems with key eviction
introduced in O. Specifically, many system activities (e.g.
ChooserActivity, activities in Settings, etc.) are marked direct-boot
aware, and therefore can be started while the work profile is locked
with key evicted. Currently they either bypass the keyguard when they
should not, or simply crash due to profile still being locked.

In the future, we need to create a new mechanism to allow activities
such as video calls, alarm clocks, etc. to bypass the work keyguard. It
probably involves checking for something like FLAG_SHOW_WHEN_LOCKED.

Bug: 36961785
Bug: 35708183
Bug: 30296144
Test: manual, by following the steps in the bugs quoted
Test: runtest -c com.android.server.am.ActivityManagerServiceTest frameworks-services
Change-Id: I5ccaaf963f3dd96e4abb785a10aa258b15363178

Merge "Fixing content intent on External Storage notification" into oc-dev

[DO NOT MERGE] Allow multiple toasts for the focused app

We don't allow apps to add multiple toast windows to
prevent an attacker to keep adding the same toast as
a workaround for our measure to ensure toast windows
are removed after a timeout. The may cause backwards
compatibility issue for apps that add multiple toasts.
While we need to fix the security vulnerability it is
desirable to make the fix as backwards compatible as
possible. This change allows the focused app to add
as many toast windows as it wants since they will be
removed after the timeout and once the app is not the
one the user uses it will lose the multiple toast add
capability.

bug:30150688

Change-Id: I2d9295926cb49b5bb80c7af2546872ff8ca22c64
(cherry picked from commit 296a60acc3d67cea23bae167dbcb51c0d0d60b23)

Clear calling identity before verifying listeners

Also don't throw when can't verify listener.

And update mocking in tests to clean state between tests.

Bug: 36783632
Fixes: 37263567
Test: runtest-systemui-notification, create a secondary user
Change-Id: I5ec95539c9859b67b8fbc7e6a85334e08e6b5a98

Merge "Add hidden SystemApi for two setup wizard actions" into oc-dev

Updates reportChooserSelection to be synchronized with queryUsageStats
in UsageStatsService, to fix a flaky test.

Bug: 35763877
Test: manually ran the flaky tests for 50 times, to verify that it is
stable.

Change-Id: I56d355bb283275f33bf7a24d0cc5b6454b99a35e
(cherry picked from commit 434f76a8d739060b5c8e96b4824687c19b94ee79)

Merge "VolumeShaper: update API, use "Millis" instead of "Ms"" into oc-dev

Fix regression when removing task.

- Limit removing the task immediately to when there is a single task
  overlay activity. Certain activities may not work when force-removed
  before the clean up work in removeTask() completes especially if the
  finishing activity is a part of a launch process (ie. trampoline
  activity). This behaviour is only needed for finishing activities that
  have actual task overlays.

Bug: 36761629
Test: Launch SiriusXM, and check that it loads
Change-Id: I34c812ab111819e858fe36b6359f82da717ba459

Merge "Show x for short period of time when activity is PIP'd" into oc-dev

Merge "Optimize HW_AV_SYNC playback by avoiding multiple timestamp block allocations." into oc-dev

Binder: Be forceful about a forceful exit.

We were previously using exit(1) when code servicing an IPC threw
any subclass of Error. That made it much harder to diagnose cases
where that happened because :

- exit runs global destructors, which might prove problematic (see
  linked bug).
- such exits are often due to bugs in application code (things like
  AssertionErrors being thrown) but aren't flagged as such by our
  infrastructure, or by humans for that matter.

To address both issues, use FatalError() so that the runtime can dump
more useful information to the logs before it aborts.

Test: manual
Bug: 36813403
Change-Id: I5826090229109dc7cb19f0c3571c609f990cd36a

Merge "jni: isLoggable: adapt to removal of property name size limit" into oc-dev

Introduce FILE_ID for TTC fonts and variation fonts.

A single TTC font or variation font can be used for multiple entry
of FontResult. To share the file contents, assign same URI for those.

Bug: 36494487
Test: android.provider.FontsContractTest passes
Change-Id: Ibf24f216179a6481dee1801cd2dfb68c4bb38fac

Merge "Always request keyboard shorcuts, even if there's no menu." into oc-dev

Update Typeface.Builder API to reflect API feedback.

- Stop pooling and remove reset method.
- Introduce overloaded constructors instead of setSource methods.
- Make italic parameter boolean and treat "not called" as
  RESOLVE_BY_FONT_TABLE.
- Remove constants and move them to Typeface with private static
  constant.
- Fix JavaDoc.

Bug: 37082657
Test: android.graphics.cts.TypefaceTest passed
Change-Id: I0c50cf3e2d50c0021f7e12bbb7ed7ae89fa1c2a1

Rename WEB_ACTION_ENABLED to INSTANT_APPS_ENABLED

Renames the instant apps setting to reflect what it is for.

Adds the SystemApi annotation to make this field visible
to the resolver and installer.

Test: existing tests

Change-Id: I1651bb101d69bdfdaa63c004435025c68a33cd8e