Merge "Catch WindowManager exceptions so they don't crash the system." into oc-dev

Merge "Handle renamed packages during default grants" into oc-dev

Merge "Fixed an issue where would could be left with empty notifications" into oc-dev

Merge "Disabled task snapshot for TV" into oc-dev

Update AccountManagerService side logic for .getAccountsByTypeForPackage
with null type.

Bug: 37987384
Test: manual, APCT
Change-Id: I4d5f735736f1b1d7bbf778aee2ec813071c34f19

Merge "Add state machine model for skipping initial brightness ramps." into oc-dev

Merge "Remove usages of ActionBarActivity." into oc-dev

Handle renamed packages during default grants

If a device has a privileged app on the system image which was
updated and a future OTA renames the app package (supported only for
privileged apps) we end up with a disabled system package setting
with no package information. Since we are not doing a null check
of the package we get from the disabled package settings during
default grants we crash leaving the system in a bad state.

Test: Manual
Change-Id: Id7255c92a4a52e368266739cd331977256eee2d3
Fixes: 62376260

Merge "Fix #62350081 - annotate permission requirements" into oc-dev

Fix issue with dialer launch transition

Test: Disable assistant, launch dialer on secure keyguard
Test: Launch assistant
Test: Launch camera

Change-Id: Ic3231f5f28fa0cf704b5f5d6e1993b296f14f5a3
Fixes: 38450585

Merge "Associate freezing display with a target display id." into oc-dev

Disabled task snapshot for TV

This was causing memory leaks when opening/closing Apps view. Will
disable this feature on TV since we are running low on RAM.

Bug: 62040693
Bug: 62359697
Test: go/wm-smoke
Change-Id: Ida290e8babc83a998a55efd10d602eb7f05f3102

Merge "Add back recents staggering entrance animation" into oc-dev

Merge "Don't show unset score values for AccessPoints." into oc-dev

Merge "Fix visibility issue in AccountManager.getAuthTokenByFeatures." into oc-dev

Merge "Don't allow removed DividerView to resize stack" into oc-dev

Don't show unset score values for AccessPoints.

Merged-In: I0fd96da6a06dafd8c8ebb435a2725ab155bd26ed

Bug: b/3692675
Test: Visual confirmation
Change-Id: I52909bf55e6efabd2be6c17542f2a15135d25c62

Merge "Refer to ShortcutManagerCompat in the javadoc" into oc-dev

Don't allow removed DividerView to resize stack

If a divider view is removed or in the process of been removed, then it
should no longer have any influence on the size of stacks.

Fixes: 62273599
Test: go/wm-smoke
Test: enter split-screen and make sure recents is visible, lock and
unlock the device and verify things look fine.

Change-Id: I79aca82be8a7013328b78e82f508c4ed9fa48a4a

Merge "Give Doclava our manifest; more permission docs." into oc-dev

Remove usages of ActionBarActivity.

ActionBarActivity has been deprecated for 2+ years and has been
extending AppCompatActivity for all that time. These changes
should be no-ops.

Bug: 35253513
Test: None
Change-Id: Ica46ccbc2b29a6a3399f7deed6045c1f7e1cd429
(cherry picked from commit 19bda3d9176b02198df4cec807d86e6797f8993e)

Associate freezing display with a target display id.

Currently, start/stop freezing is always associated with the default
display. As a result, any animations that result from freezing will
occur on the default display. One situation this happens is during a
configuration change, where a screen rotation can be triggered. In
the case of adding a secondary display, a configuration change is
always triggered to apply the override configuration at the display
level. As a result, the default display exhibits a rotation animation
upon unfreezing during surface placement.

This changelist passes down the display id associated with the action
so that the animations may be applied to the right display. In the
scenario described perviously, the secondary display id is passed down
and therefore is the target of the rotation animation during
unfreezing.

Fixes: 35486733
Test: go/wm-smoke
Test: cts-tradefed run singleCommand cts-dev --module CtsViewTestCases --test android.view.cts.SurfaceViewSyncTest#testSmallRect
Change-Id: I87e56f5901934327fc50b3c7e543b5e36b8ff34f

Merge "WifiConfiguration: add local only networkId" into oc-dev

Merge "Coordinate configuration changes and window frame size." into oc-dev

Coordinate configuration changes and window frame size.

Previously, it was possible for the view hierarchy to be measured,
laid out, and drawn around a window frame size that did not match
the current configuration. This stems from new configurations not
always propagating back from WindowSession#relayout, which is
called from ViewRootImpl.

This changelist makes WindowManagerService#relayoutWindow always
return the latest configuration. It also adds rotation to the
configuration.

Fixes: 32839232
Test: go/wm-smoke
Test: Open Camera while rotating phone to landscape. Added
      temporary logs to detect inconsistencies between measurements
      and reported rotation on draw.
Change-Id: I39daca338b4f87eff1a509eb99493f01e710ced1

Merge "Revert "Disabled task snapshot for TV"" into oc-dev

Revert "Disabled task snapshot for TV"

Bug: 62360140
Bug: 62040693

This reverts commit 4256422eb6752294a08f4701dd66798652a8059f.

Change-Id: Id89de52c9ad25ef7b3163bd8b4a0c44f97631431

Merge "Don't verify instant apps" into oc-dev

Fix #62350081 - annotate permission requirements

The Alarm Manager APIs that allow callers to ascribe wakelock time
to other apps are gated on the UPDATE_DEVICE_STATS permission.
Document this with the appropriate annotation.

Test: N/A
Change-Id: Idf5548089fb7d2235139ae8ba7bf58e7f455eb33

Merge "Add ERI (enhanced roaming indicator) 66 to list of home ERIs for Verizon" into oc-dev

Merge "Disabled task snapshot for TV" into oc-dev

Merge "Reinstate SurfaceView Background from N" into oc-dev

Give Doclava our manifest; more permission docs.

We've seen some @SystemApi methods protected with non-system
permissions, so give Doclava the platform AndroidManifest.xml so it
can parse the actual permission protection levels to look for APIs
that are letting in non-system apps.

Also document more @SystemApi permissions.

This is purely a docs change; no logic changes are being made.

Test: make -j32 update-api
Bug: 62263906
Change-Id: Ie0f0a5fb0033817bcc95060f2183a52ae4ae7b06

Merge "Fix icon tint on notification shelf" into oc-dev

Merge "Initialize last-orientation to ensure we don't skip loading config." into oc-dev

Merge "Annotate @SystemApi with required permissions." into oc-dev

Merge "Revert "Fix race condition triggered by quick stop/start of Dream"" into cw-f-dev am: 7a5dd1c3ff
am: 51a89ee90c

Change-Id: I982b94c155849061a667b1bf90f7a511bb81b1c5

Merge "Revert "Fix race condition triggered by quick stop/start of Dream"" into cw-f-dev
am: 7a5dd1c3ff

Change-Id: Ida35556f6d57113ad06c736ef780580b814f673e

Merge "Fix tinting of guest icon in QS footer" into oc-dev

Merge "Revert "Fix race condition triggered by quick stop/start of Dream"" into cw-f-dev

Merge "Ignore entries for isolated uids if there's no mapping for them." into oc-dev

Add back recents staggering entrance animation

Since there was a change in ValueAnimator that did not allow setting
initial play time per interpolator grouped in an AnimatorSet, set the
initial play time for each animation to allow the same effect.

Change-Id: I05445bde6e185a99250d7d193bd0e74d5ce98fa0
Fixes: 36175824
Test: manual - launch recents

Fix visibility issue in AccountManager.getAuthTokenByFeatures.

Bug: 34679769
Test: manual, APCT
Change-Id: I7270a5a5dd681ad088fa28e48d849ba3541e45cd

Merge "Fix up some jank from unnecessary reinflation" into oc-dev

Annotate @SystemApi with required permissions.

Most @SystemApi methods should be protected with system (or higher)
permissions, so annotate common methods with @RequiresPermission to
make automatic verification easier.

Verification is really only relevant when calling into system
services (where permissions checking can happen on the other side of
a Binder call), so annotate managers with the new @SystemService
annotation, which is now automatically documented.

This is purely a docs change; no logic changes are being made.

Test: make -j32 update-api && make -j32 offline-sdk-docs
Bug: 62263906
Change-Id: I2554227202d84465676aa4ab0dd336b5c45fc651

Merge "Added mising sendToTarget message for dismissing docked stack" into oc-dev

Disabled task snapshot for TV

This was causing memory leaks when opening/closing Apps view. Will
disable this feature on TV since we are running low on RAM.

Bug: 62040693
Test: Compiles
Change-Id: Ifc43118dd072f17a047003d1797004bd441669b8

Merge changes from topic 'dungeon' into oc-dev

* changes:
  Hide redundant foreground service notifications.
  Updates to Dianne's Dungeon.

Merge "Remove ptp from persist config on boot" into oc-dev

Don't verify instant apps

Change-Id: I07bc45bc82d1cbd2012c4605d5ca19f2ef36d615
Fixes: 62202471
Test: Manual; install instant app, see verification is disabled
Test: Manual; install full app, see verification is enabled

Ignore entries for isolated uids if there's no mapping for them.

Bug: 25195548
Test: manual
Change-Id: If5aa7e194f4d43f2a7068899b9aa6584bfb2b7cc

Merge "DO NOT MERGE: AAPT2: Allow undefined resources (placeholders)" into oc-dev

Fix tinting of guest icon in QS footer

Test: visual
Change-Id: I1daaac94e6bbff2e94583e1501c43beb4d9f757f
Fixes: 62339982

Fix up some jank from unnecessary reinflation

Don't need to re-inflate fragments because of ui mode change.

Test: manual
Change-Id: I8cb8ff4f64b5c1cb5982d39c6acced682eaa4d41
Fixes: 62041537

Revert "Fix race condition triggered by quick stop/start of Dream"

Causes another race condition

This reverts commit 69515b2261335be4bd8510ed5a40a6bed3d7b48f.

Bug: 62147987
Change-Id: Ibe6f9bc14ed6cc35d3a23d52300657eaed9bb6d8

Merge "Protect instant apps from full apps" into oc-dev

Merge "Annotate locks" into oc-dev

Hide redundant foreground service notifications.

If an app with a foreground service has (at least one)
FLAG_FOREGROUND notification shown to the user, we allow
that to satisfy the requirement that the user be informed
about such things. But if the fg notification or its channel
is blocked by the user, we show the NOTE_FOREGROUND_SERVICES
notification (a/k/a Dianne's Dungeon) provided to us by the
activity manager.

Note that if even one of the foreground processes for the
current user is missing its disclosure notification, the
user will see the whole dungeon.

Bug: 36891897
Test: runtest -x frameworks/base/packages/SystemUI/tests/src/com/android/systemui/ForegroundServiceControllerTest.java
Change-Id: I4f5d96f80b7c1901faadb56661a42d26f746aa88

Updates to Dianne's Dungeon.

 - new EKG icon
 - set the notification timestamp to the oldest service
   start time, so you have some idea how long things have been
   this way
 - minor text fixes

Bug: 36891897
Test: runtest -x cts/tests/app/src/android/app/cts/ServiceTest.java
Change-Id: I99db280cde8ca3ecd7205cd44fac159d8f652ca2

Added mising sendToTarget message for dismissing docked stack

Toast that an app doesn't support split screen wasn't displaying because
of missing line to actually send the
NOTIFY_ACTIVITY_DISMISSING_DOCKED_STACK_MSG message.

Test: Dock an app and the launch an other app that doesn't support
split-screen and make sure the "doesn't support split-screen" toast
shows up.
Test: go/wm-smoke
Fixes: 62281412

Change-Id: Ia2cb3085b6c6bfd27cf9e825315b8ddc89cf5bcd

Merge "Don't allow blocked apps to post notifications" into oc-dev

Fix captive portal login ssl error overflow

This patch fixes the built-in html page shown in the captive portal
login activity when there is an ssl error. The page is now scaled
correctly with respect to the device screen, and text wraps as
expected.

This patch also cleanups the hardcoded page by:
 - formatting it for better reading,
 - inlining the "px" css rules,
 - using string concatenation instead of format for including the
   localized text.

Bug: 62269512
Bug: 36532213
Test: manually triggered the activity to load the error page
Merged-In: I9d037222798bd9d4c1736923afb71e3e14c68bcc

(cherry pick from commit 60d5f46d89038a6a46c38ac0e57d0762a67732cb)

Change-Id: I5cb6e53f5af9dc4b93be8da1d9b6e0e8958e561f

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

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

Merge "Move strings to SettingsLib so TV can use them" into oc-dev

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

Merge "Workaround to ensure that PIP activities have a visible shadow." into oc-dev

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Import translations. DO NOT MERGE

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

Merge "Add media.codec to native stack dump list" into oc-dev

Merge "Handling cases with a null bitmap." into oc-dev

Merge "Workaround glitches from SOFT_INPUT_ADJUST_PAN" into oc-dev

Merge "Fix issue with jumping PIP when IME shows." into oc-dev

DO NOT MERGE: AAPT2: Allow undefined resources (placeholders)

A resource defined like so:

<item type="drawable" name="foo" />

should be assigned the value @null.

The only exception is for <string> resources, which are given the
empty string value (since <string></string> is ambiguous). The decision
to use "" is based off the fact that old AAPT used to assign "" to all
undefined resources, even non-string ones.

Bug: 38425050
Test: make aapt2_tests
Change-Id: Ib3e0f6f83d16ddd8b279c9fd44a07a37867b85e9

Add media.codec to native stack dump list

This is still needed for non-treble devices.

Test: do 'adb shell bugreport' on angler, and check that
'Cmd line: media.codec' and the stacks should be there.

bug: 62297059

Change-Id: Ic46b044e471a2ec3c432bb2fdc6afd6fb1f8090e

Workaround to ensure that PIP activities have a visible shadow.

- When the window for the activity enters PIP, update the outline provider
  to override the alpha of the shadow (to be opaque) to ensure that is is
  visible.  Only applies to the task root activity.

Bug: 36741700
Test: Launch YT, ensure that there is a shadow when after it enters PIP
Test: go/wm-smoke
Test: android.server.cts.ActivityManagerPinnedStackTests

Change-Id: If089dae84e4916d3d0e7bbeb316215b46e522e05

Fixed an issue where would could be left with empty notifications

If the state between systemui and the notification manager got
out of sync, we would not immediately remove the notifications when
using clear all.

This could lead to holes in the notification shade
temporarily. While this isn't a fix, it helps avoiding these ugly
states, similarly to how we already do it when manually swiping.

This also improves that only notifications that are visible are
actually participating in the clear all animations instead of
all of them.

Test: manual, hit clear all observe animations / normal clearing
Bug: 62171447
Change-Id: I83d0f3cb0bae4bc43bf35d3f9399cebc05259680

Workaround glitches from SOFT_INPUT_ADJUST_PAN

Change-Id: I1ee0c7d434eb03667bd5e838ea891db73651a994
Fixes: 62073166
Test: Repro app in bug

Merge "add channel and group tags to the notificaiton logs" into oc-dev

Move strings to SettingsLib so TV can use them

Bug: 38496265
Test: Compiles
Change-Id: I68b535ba490e979a1869b7ea7e2be7737aaaff95
Merged-In: I68b535ba490e979a1869b7ea7e2be7737aaaff95

Merge "Notify keyguard of when power button is pressed." into oc-dev

Catch WindowManager exceptions so they don't crash the system.

WindowManager throws an IllegalStateException when adding a view twice; this
should never happen (since show() and hide() are always called in the UIThread),
but when it does, it should not crash the system.

Test: existing CtsAutoFillServiceTestCases pass
Fixes: 62262781

Change-Id: Idfd246ff173f4abb606865b7ba8cc367eb6acaad

Handling cases with a null bitmap.

- This can happen either if an app creates an ActivityOptions without a
  thumbnail, or if the call to create a hardware bitmap fails for any
  reason. Just ignore the thumbnail for the transition in this case.

Bug: 62296016
Test: Have not been able to reproduce, but this is just a logical change
Change-Id: I30776b651df1f42118fe1d317fa4817261a6e977

Reinstate SurfaceView Background from N

In N we used a Dim-Layer to add a background to SurfaceView, from
the WM side. In O we forgot to reimplment this with the new SurfaceView
and so we once again can have holes for apps which don't implement
surfaceRedrawNeeded.

Bug: 62113351
Test: Manual from bug. go/wm-smoke
Change-Id: If3dac51886b9a8083140da7b5bc1b349da57860f

Merge "Avoid excessive logging when the necessary proc file is unavailable." into oc-dev

Merge "wifi: add utility function to return saved configurations" into oc-dev

add channel and group tags to the notificaiton logs

Change-Id: Icf029e42fca46262c9d9f8ff63f26de098f888bf
Fixes: 37714086
Test: runtest systemui-notification

Protect instant apps from full apps

Bug: 35871369
Test: cts-tradefed run commandAndExit cts-dev -m CtsAppSecurityHostTestCases -t android.appsecurity.cts.EphemeralTest
Test: Manual; install instant app and ensure it runs
Change-Id: I1835fe58256377f901db38fcc3e4c98886d799ac
Merged-In: I1835fe58256377f901db38fcc3e4c98886d799ac

Merge "Use a PopupWindow to show tooltips" into oc-dev

Notify keyguard of when power button is pressed.

This is done on wear power button doesn't turn off the screen,
when the device wakes from keyguard UI isn't visible yet, so
it needs to react to power press in some way.

Bug: 35147955
Change-Id: I22619ea446770d09b53370e9244215646b60a9db

Merge "QS vis tweaks" into oc-dev

Remove ptp from persist config on boot

We got a report from a user in which ptp was set in the
persistent config, likely from a previous version.
This causes errors in the usb state and is not removed
by an ota. To fix, remove ptp in the same place that mtp
is removed from the persistent config.

Bug: 62202885
Test: Add ptp to persistent config, verify removed.
Change-Id: I5ebd93b9c8a49bcaca5a3362e49ed1e1acf50a9b

Avoid excessive logging when the necessary proc file is unavailable.

Bug: 34133340
Test: manual
Change-Id: Ic00e58a5b7bb7f92a0467df1cab8df5cab7dffda

Add state machine model for skipping initial brightness ramps.

Bug: 36092576, 62293899

Change-Id: I7fbcd6989b938f0bc8d369c263116e9d374ff0ee

Merge "Track added/removed window when focus is null" into oc-dev

Merge "Improve consistency in focusAfterDescendants behavior" into oc-dev

Merge "Annotate test method visibility in WindowManager" into oc-dev

QS vis tweaks

 - QS Header status centering 24dp -> 32dp
 - Several fonts condensed -> medium

Test: visual
Change-Id: Ic8bc08d6a4d78ee708c4a890140b87716a883373
Fixes: 38199857

wifi: add utility function to return saved configurations

The saved configurations included saved Wi-Fi networks and
Passpoint profiles.

Bug: 62048284
Test: manual test
Change-Id: I55be798d5f36fa678bf69d9f910ca10ed952518a