Use Empty List for Default PhysicalChannelConfig

If a caller registers to listen() for a physical
channel config message, the beavior is to immediately
trigger an unsol. In the case that a list hasn't been
sent from the modem, we should send a default empty
list rather than a null. This is the preferred
approach for Android APIs because it reduces the
reliance on null checks by callers.

Bug: 78536847
Test: ¯\_(ツ)_/¯
Change-Id: I9e59195f5d64264ab473b2fd7e4f7eb6446c4829

Merge "Rounded corners can be different on top/bottom" into pi-dev

Merge "Flip task animation direction" into pi-dev

Merge "Revert "Reset SIM state if the subscription is no longer active."" into pi-dev

Merge "Always copy SignalIconState when keeping a reference" into pi-dev

Merge "Rework Exception Handling for IpSecManager" into pi-dev

Merge "Update comment for b/75318890 TODO" into pi-dev

Rounded corners can be different on top/bottom

Test: manual, visual inspection
Bug: 75949920
Change-Id: I1e9a9df0767aa5235e67b752a19c8e4b32701ed1

Merge "Temporarily allow StackView to use a canvas.clipRectUnion" into pi-dev

Merge "Add config value for fingerprint gesture support" into pi-dev

Merge "Copy links to orginal text in a11y node info" into pi-dev

Merge "A11y isHeading takes old API into account" into pi-dev

Revert "Reset SIM state if the subscription is no longer active."

This reverts commit b1b24e71f55de2bfa2cc8d0ff96ec0ac021e663d.

Reason for revert: This commit will cause side effects with Project Fi. It'd be too risky to solve this issue at the current stage of Android P. Revert first.

Bug: 77579824
Bug: 77702165
Change-Id: I07035a2854427021ac0bfaa9c696daabefe6fd64

Merge "Revert "Reset SIM state if the subscription/slot is no longer active."" into pi-dev

Merge "DO NOT MERGE Make "Want to start battery saver?" dialog better." into pi-dev

Merge "Ellipsize long QS labels." into pi-dev

Update comment for b/75318890 TODO

Bug: 77862563
Bug: 75318890
Test: N/A - comment change
(cherry picked from commit 2333f7cba65a0fc2036bcf0e0b2611ad114d17f7)

Change-Id: I52c9e37e4629d740cb04d552bc967d6b8ec5fba3
Merged-In: Ie48f1df394ccae5420a18260a4eaefe94d84497d

Always copy SignalIconState when keeping a reference

Defensively copy SignalIconState so that StatusBarWifiView and
StatusBarMobileView can properly detect changes to state.

Also don't do visibility changes if setting the same visibleState on
StatusBarWifiView.

Test: turn wifi on and off; verify that it goes away properly in status
bar
Fixes: 78353558
Fixes: 77738471
Fixes: 78050495
Fixes: 78163313

Change-Id: Ie9b9e05885c130c1ec6d6abb9d0567400df23e52

Merge "Keyguard: fix possible divergent state" into pi-dev

DO NOT MERGE Make "Want to start battery saver?" dialog better.

- Don't line-break in "Battery" (only in English)
- Add "Learn more" link.

Bug: 78261259

Test: Manual test with:
- adb shell dumpsys battery unplug
- adb shell settings delete secure low_power_warning_acknowledged
- Enable battery saver
- Make sure the link is clickable if a link is set.
- Make sure there's no "learn more" link if the link is not set in strings.xml

Change-Id: I83364f628dd596a4d50bf2aca4db7cbfe7cf4909

Merge "Move brightness color conversion from SysUI to settingslib" into pi-dev

Merge "Synchronize RootWindowContainerTests" into pi-dev

Merge "Potentially fix notification launch animation" into pi-dev

Merge "Add new hidden APIs" into pi-dev

Merge "Add more light greylist entries." into pi-dev

Merge "Added OWNERS file to am package." into pi-dev

Add new hidden APIs

Bug: 78552452
Test: make
Change-Id: I73dc97a33b645f6cd9b7bf4a37254737ed3107ce

Keyguard: fix possible divergent state

Fixes an issue where the state in SystemUI and AM/WM could
get out of sync: If the keyguardGoingAway transition is
started and then later canceled by SystemUI, it never
tells AM that it canceled it.
This causes SystemUI to show itself in keyguard mode,
meanwhile AM/WM do not hide the underlying app because
the status bar is in the going away transition as far
as it is concerned.

Change-Id: I5966f0655fd694ae599a83dda7a9f6463088632b
Fixes: 76367931
Test: delay keyguard going away callback, unlock phone, during the delay, remove SIM; verify it does not end up in bad state

Merge "New light grey APIs." into pi-dev

Merge "More light grey APIs." into pi-dev

Merge "Add more light grey APIs." into pi-dev

Merge "Remove unnecessary WTFs from password state check" into pi-dev

Add more light grey APIs.

bug: 64382372
Test: m
Change-Id: Ib15b65c376e4038232d4c1e04d10e508b34786af

More light grey APIs.

bug: 78436346
Test: m
Change-Id: I8bff5332abc93d49170c77fabb589bbcfa9249b6

New light grey APIs.

bug: 78471731
Test: m
Change-Id: I52f34f735873de485a73169f1ebbffe9484d0fb7

Merge "Refactor GnssGeofenceProvider" into pi-dev

Merge "Extract interface for the TemperatureView" into pi-dev

Rework Exception Handling for IpSecManager

In order to properly support EOPNOTSUPP this CL
applies a consistent approach to handling Exceptions.
Hereafter, all exceptions that aren't of a special
method-specific type (such as SpiUnavailableException)
will all be returned to the calling process unchanged.
At the API call site, the ServiceSpecificException,
which is really an Errno, will be inspected and either
converted to an unchecked exception for types we know,
or it will be converted to an IOException in cases where
that method can return a checked exception. In cases
where we do not expect an errno, we will simply throw
a generic RuntimeException. This means all API calls
will now properly throw UnsupportedOperationException
and may be CTS tested accordingly.

Bug: 72420898
Test: runtest frameworks-net
Change-Id: I4a00e221618896223fcdb4b4279fb14cd14e34d8

Added OWNERS file to am package.

Why? Because I don't want to worry about things like ag/3760423

Test: N/A
Bug: 0
Change-Id: I96c38d8dfbb8ac5d530251de90acc8499f851075

Merge "Revert "Revert "Remove the OEM hook implementation and usage""" into pi-dev

Revert "Revert "Remove the OEM hook implementation and usage""

This reverts commit 7cfc5cda2f2a382a3eabd1c5382c412f2d1dedcf.

Reason for revert: Causes a critical error on Cuttlefish.

Bug: 78299964

Change-Id: Icb06aea5eb42fd156a967c4d4e39aefc51d91cf4

Merge "Allow for the lock screen to have a different set of nav buttons" into pi-dev

Synchronize RootWindowContainerTests

There were a few failures that seem to be threading issues.
Specifically, IndexOutOfBounds within a for loop that was iterating the
size of the ArrayList and ArrayMap index failure. RootWindowContainerTests
calls into methods in WM that assume the WM lock is already held. Ensure
testSetDisplayOverrideConfigurationIfNeeded is synchronized.

Test: RootWindowContainerTests#testSetDisplayOverrideConfigurationIfNeeded
Change-Id: I5003fc4a59baae4990a1464dbbc85b0938312994
Fixes: 78349285

Merge "Fix destruction of preserved surfaces for non app windows." into pi-dev

Merge "Don't call into AM finish-recents-animation while holding onto the WM lock" into pi-dev

Merge "Actually @hide unwanted capability methods." into pi-dev

Merge "Added OWNERS file for wm package." into pi-dev

Merge "Attach color space after performing a readback" into pi-dev

Merge "Fix rounding of padding in NinePatchDrawable" into pi-dev

Merge "BluetoothManagerService: Add dumpsys logs for BLE_ON -> OFF" into pi-dev

Merge "RRO: Update target overlays after install" into pi-dev

Added OWNERS file for wm package.

Why? Because I don't want to worry about things like ag/3760423

Test: N/A
Bug: 0
Change-Id: I230b7e8aead53395da5c58aa27ed547cf06fb327

AudioPlaybackConfiguration release/death: notify listeners

When a player dies or is released, dispatch an update to the
  AudioPlaybackConfiguration listeners.

Bug: 70195346
Test: atest AudioPlaybackConfiguration#testCallbackMediaPlayerRelease
Change-Id: Id0b62abb32879d3166da26f20a334d0447183ac0

Merge "Don't automatically decline tracking jobs with met initial delays" into pi-dev

Allow for the lock screen to have a different set of nav buttons

Bug:74446022

Test: Manually. Test by switching users that have PINs set

Change-Id: Id956b0cfe76eba82235b82080c755489bdf67938

Move brightness color conversion from SysUI to settingslib

The color space conversion logic in BrightnessController is also needed
by Settings app to display brightness percentage correctly. This change
moves the conversion logic to settingslib so it can be shared by other
UIs such as Settings/VrSettings.

Bug: 73810208
Test: atest RunSettingsLibRoboTests
Change-Id: If92f0da2d930fec797902b5d1b3731100e1d0ea4

Ellipsize long QS labels.

Since marqueeing only works when a TextView is a single line, make the
label a single line if the layout would otherwise require 3+ lines.

Change-Id: Ib1f47145e01696b01cdce893fe96eca8fbe2ff99
Fixes: 78345638
Test: visual

Remove unnecessary WTFs from password state check

It is possible to have null PasswordMetrics object for a given user -
if, for example, the user never had a password set.

Do not WTF in that case.

Bug: 78191197
Test: That it compiles.
Change-Id: I807c2755890b0772e295b4cb0095cac1bf2d0aef

Merge "Use log.wtf on debug devices if property cannot be set" into pi-dev

Merge "Move status bar accessibility title to pane" into pi-dev

Copy links to orginal text in a11y node info

Not doing this copy results in us keeping
mOriginalText around. That is a CharSequence that
can contains Spans that reference other Views and
other expensive stuff.

Fixes: 78511639
Fixes: 75602764

Test: make
Change-Id: I977646311167f8d13e1c4a5c8fc38372e6d1ff3c

Merge "config.xml: Add config flag for filtering DBS scans" into pi-dev

RRO: Update target overlays after install

Newly installed apps were not having the their set of enabled overlays
set correctly. This allows all newly installed packages to update their
set of enabled overlays in the package manager.

Bug: 77279210
Bug: 65319218
Change-Id: I82a10e96e9869bb6fc1ad00305ec4b6cfb17483b
Test: Manual validation of b/77279210 and duplicates. Also enabled a previously failing
PTS test in ag/3941515 within the same topic.

Merge "Fix NPE when deciding whether to defer task stack animation." into pi-dev

Merge "Add presubmit hook for hidden API lists." into pi-dev

Merge "pm: skip scan-as-privileged for OMR1 vendors" into pi-dev

Add more light greylist entries.

Scraped from bugreports by the compat bugbot module. See
go/hidden-api-colab for more details.

Test: m
Bug: 74410240
Bug: 77459362
Bug: 77927354
Bug: 78084664
Bug: 78371474
Bug: 78433404
Bug: 78436346
Bug: 78438386
Bug: 78467671
Bug: 78471731
Bug: 78483748
Bug: 78484047
Bug: 78485081
Change-Id: Ie79f892d5cdccef442944286cc718a079bc3031f

Potentially fix notification launch animation

If we don't have something to animate, immediately finish
animation

Test: Open a couple of notifications
Bug: 77847366
Change-Id: I06c651b33a5ffb4ec236ba41f9db38b0125684dd

Add presubmit hook for hidden API lists.

Check that they're sorted as expected and contain no duplicates.
The sort order now uses:
$ LC_COLLATE=C sort -f

So that non-alphanumeric characters are not ignored, giving a more
intuitive sort order. the '-f' means ignore case.

Also sort the existing lists accordingly.

Test: repo upload
Bug: 64382372
Merged-In: I52b884da33a9a46455df6747a215683d9d3c3218
Change-Id: I4cdd3bc5c11be91a9a3f678580af49ac67f3c968

Merge "Keyguard: Don't lock when phone process crashes" into pi-dev

Merge "Add more hidden APIs to light greylist" into pi-dev

Merge "Edit dnd icon in volume dialog" into pi-dev

Merge "Make password history hashing more secure" into pi-dev

Merge "update wifi unit test script" into pi-dev

Flip task animation direction

In order to be consistent with Recents order in Launcher.

Furthermore, we also make RTL consistent with Launcher order,
which is reversed in RTL.

Test: Compose email in Gmail
Test: Compose email in Gmail in RTL
Bug: 78231551
Change-Id: I3776b78703f8c81f768617f21d5850335a0b9788

Keyguard: Don't lock when phone process crashes

Fixed an issue where the keyguard locked in response to the
phone process crashing. Since that's not avoidable for the
moment, don't treat unknown sim states as absent.

Bug: 78060670
Test: adb shell kill `pid com.android.phone`; verify that the phone did not lock
Change-Id: Ic5aa51d3b0521740b828d934c6653f8d5ae6380a

Merge "Check self permission in BackupManager#isBackupServiceActive" into pi-dev

Merge "Only allow using PendingRemAnimRegistry on top launched activity" into pi-dev

Merge "Clarify SystemUpdatePolicy" into pi-dev

Add more hidden APIs to light greylist

Bug: 78242776
Test: make
Merged-In: I3e1f6acd805ca73509a896066a336432f29d0c87
Change-Id: Ieb2eb62201a6f270afd76ae3f6e5ca6226dd57a6

Check self permission in BackupManager#isBackupServiceActive

Old method only checked for IPC caller, method isn't an IPC, adding
check for self permission as well.

Bug: 78467264
Test: atest com.google.android.gts.devicepolicy.DeviceOwnerTest#testBackupServiceActive
Change-Id: If9eed90f2deaaf1a4f2376fdf1881e62ee8954fb

Merge changes I0cd83a93,I03ae2bbb into pi-dev

* changes:
  Make sure getActiveNetwork is consistent with default callbacks
  Add tests for getActiveNetwork.

Merge changes from topic "notification_reply_action" into pi-dev

* changes:
  Cleaned up the paddings of the messaging layout and smart replies
  Changed the size of the largeIcon
  Moved the disabling from alpha to manual color blending
  Updated the color of the reply button to be more neutral
  Updated the reply icon
  Disabled reply action when pending intents are cancelled
  Split the reply icon permanently from the right icon

Merge "Add spinner for smart replies." into pi-dev

Make sure getActiveNetwork is consistent with default callbacks

Bug: 77737389
Test: runtest framework-net
      new test don't pass without the main code change, but they
      do with it
Change-Id: I0cd83a935ab0b349aa47e065b830e5a43ab9a091

Merge "Add light grey APIs." into pi-dev

Merge "Log package name when showing targetSdkVersion warning" into pi-dev

Log package name when showing targetSdkVersion warning

Test: manual verification
Bug: 78332004
Change-Id: Ie91ac06e3a973edf655362b836698185a0aa7602

Merge "Check provider exists and is enabled when request location" into pi-dev

Cleaned up the paddings of the messaging layout and smart replies

Refactored the code such that the reply icon's inset was substracted
from the overall paddings to make all of them consistent. This
will also help in the future as we don't have to update those
paddings manually anymore.

Test: test all sorts of combinations with Notify
Change-Id: I8c6c9951bc73c88d95cb443c8e618c7adca7ebd3
Fixes: 78441535

Add tests for getActiveNetwork.

Bug: 77737389
Test: runtest frameworks-net
Change-Id: I03ae2bbb08559f2cd44979e291c1f5d50eb215da

Changed the size of the largeIcon

Also tunes the paddings with largeIcons more

Test: test margins for all templates, ensure nice wrapping
Change-Id: I9ace23c2a6b38fdf25e46fd55d6848b8922851a3
Fixes: 76415894

Moved the disabling from alpha to manual color blending

The Transformations don't preserve alpha currently and it's
hard to fix without the risk of introducing regressions.
We instead blend the color manually now.

Change-Id: Ib9c209295529c5e04564a3faed74160914951152
Fixes: 77811784

Updated the color of the reply button to be more neutral

The primaryTextColor wasn't a good fit, because it was
drawing too much attention being pure black. We're now
taking the default color instead. It's also used for
active permissions now.

Test: add notification, look at reply button.
Change-Id: If390c66e927a92a1115250abb7254fafe81b054c
Fixes: 72750728

Updated the reply icon

Change-Id: I0f8e91630cedefc6d9f44ded421243b719db7e6c
Fixes: 77244755
Test: observe new icon on notificaction

Disabled reply action when pending intents are cancelled

Previously the user could open inline reply even when the
action was already cancelled. This also enables listening
to pending intent cancellations.

Test: manual
Fixes: 77811784
Change-Id: I4ae164081c6abdeb60a8e78d61bf5e4f26cca1d3

Refactor GnssGeofenceProvider

Bug: 77974401
Change-Id: Id377b14f9f6c47e031a863ee9096c9b5c66ebf08
Fixes: 77974401
Test: m -j ROBOTEST_FILTER=GnssGeofenceProviderTest RunFrameworksServicesRoboTests

Actually @hide unwanted capability methods.

Bug: 77601789
Test: builds, boots
Test: make doc-comment-check-docs
Change-Id: I7a913b83ab5110047d187564b3a1b0bbe1631942

Revert "Reset SIM state if the subscription/slot is no longer active."

This reverts commit 3e7c7ca8e4faed7d0377258ec1fdd43e07083316.

Reason for revert: This commit will cause side effects with Project Fi. It'd be too risky to solve this issue at the current stage of Android P. Revert first.

Bug: 77579824
Bug: 77702165
Change-Id: I404cd5b4413ba95a26aabb5da55c565a37f06e7e

Merge "Update Java doc of GnssMeasurementEvent#STATUS_NOT_ALLOWED" into pi-dev