Adjust vol dnd icon layout width

When only one volume row is visible, adjust the
width of the volume dnd icon FrameLayout so the
icon floats on the top right.  Icon was previously
inset by padding introcued by volume_dialog_rows minWidth
attribute.

Test: visual inspection (with 1+ streams)
Fixes: 78275139
Change-Id: I8f34ba21545823ac8431396696adb4e9347b40db

Merge "QQS visual fixes." into pi-dev

Merge "Add more API accesses from bugreports." into pi-dev

Merge "Pipe the user restriction to the admin dialog" into pi-dev

Merge "[Magnifier-40] Always child of main window" into pi-dev

[Magnifier-40] Always child of main window

Previously, we were making the magnifier surface a child of the main
window unless the magnified view was a SurfaceView, in which case we
were setting the SurfaceView to be the parent of the magnifier. In
Chrome, where the magnified views are usually SurfaceViews, this caused
the magnifier to be displayed underneath the omnibox, which was a
terrible user experience when trying to magnify the first lines of text
on a page. This was because the omnibox had a higher Z than the
SurfaceView, and therefore a higher Z than all children of the
SurfaceView (including the magnifier).

This CL sets the parent of the magnifier surface to be the main window's
surface when the magnified view is a SurfaceView as well. Therefore, the
magnifier becomes a sibling of the Chrome omnibox and, by giving the
magnifier a higher Z, it ends up being rendered on top.

Bug: 77926365
Test: atest CtsWidgetTestCases:android.widget.cts.MagnifierTest
Change-Id: Ic5b5f6ca687db8b5d842f0ab20eac70f1fd2f85e

Merge "Fix CaptivePortalLoginActivity probes to bypass Private DNS" into pi-dev

Fix CaptivePortalLoginActivity probes to bypass Private DNS

Test: as follows
    - built
    - flashed
    - booted
    - runtest frameworks-net passes
    - manual testing on captive portal networks
Bug: 64133961
Bug: 77140445
Bug: 78548486

Merged-In: I749e8b5dba3b4af9ecb82bfb558c6f0e664531a4
Merged-In: Ic76d55861f385d1aecd360adda7f10142d083a47
Change-Id: Idbc4d90eeec257b36c5f374d61dea4600a0f8ebb
(cherry picked from commit 848d842a3ad94720e1d70db3337048e9568bb9d6)

Merge "Do not magnify if surface control is not set" into pi-dev

Merge "Refactor GnssMeasurementsProvider" into pi-dev

Merge "Fix WTF in PendingIntentRecord.unregisterCancelListenerLocked()" into pi-dev

Merge "Add ADAPTIVE_BATTERY_MANAGEMENT_ENABLED" into pi-dev

Fix WTF in PendingIntentRecord.unregisterCancelListenerLocked()

It's possible that detachCancelListenersLocked() has just been called
when unregisterCancelListenerLocked() is called, so need a null check.

Change-Id: If4b18d713595b1cf1a29ba05a3736892b21931a4
Fixes: 78592185
Test: build

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

Merge "Work on issue #78480444: Start tracking uid state in app ops" into pi-dev

QQS visual fixes.

- Allow QQS header status icon / date view header to expand with larger
font sizes.
- Ensure alarm icon in QQS header does not show without the
corresponding alarm text.
- Fix QQS status bar, icons and QS tile margins.

Change-Id: Icf11fc5c581cb6d3254f6223ba4bcac2879c7f16
Fixes: 78571967
Fixes: 78536526
Fixes: 77123524
Test: visual

Merge "Fix floating toolbar menu item spacing." into pi-dev

Revert "Edit dnd icon in volume dialog"

This reverts commit b36ef4eb5779a03c25d2fbeb283944f93e273583.

Bug: 78554550
Bug: 78275139
Reason for revert: Adds unnecessary spacing when there are multiple volume rows

Change-Id: I8f858b152f92418dce8f1088e892b758baaa2299

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

Add more API accesses from bugreports.

This includes APIs accessed by 'linking', but only those detected at
runtime (i.e. excluding dex2oat invocations).

Test: m
Bug: 77272253
Bug: 77533294
Bug: 77854069
Bug: 77912264
Bug: 77988118
Bug: 78265113
Bug: 78339426
Bug: 78370565
Bug: 78437948
Bug: 78438386
Bug: 78439315
Bug: 78441396
Bug: 78484047
Bug: 78484886
Bug: 78513977
Bug: 78515673
Bug: 78529635
Change-Id: I9f07b650f565562d43c50e0606ec52a37c797ce7

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

Fix floating toolbar menu item spacing.

We mistakenly changed floating_toolbar_menu_button_side_padding in
I309c729eb842d9388066bfb43eb18f33dbfe10b8 where in fact what we
wanted to do was to just change the mIconTextSpacing.
This cl fixes that.

Change-Id: I1b940eb12d2948f8c167c3f37ed0eba105b9f5ce
Fixes: 74032743
Test: built, observed visual look
Test: bit FrameworksCoreTests:android.widget.TextViewActivityTest
Test: bit CtsWidgetTestCases:android.widget.cts.TextViewTest

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

Pipe the user restriction to the admin dialog

This change allows us to show different strings for different user
restriction.

Instead of setting DevicePolicyManager.EXTRA_RESTRICTION everywhere
in Settings, EnforcedAdmin object now stores the restriction. So when
we are starting the admin dialog with the EnforcedAdmin object,
we can set the EXTRA_RESTRICTION bit.

Test: make  RunSettingsLibRoboTests
Bug: 78443458

Change-Id: Iabc1629b9ed18f881cdc8c15ec5e49a4b50f76a4

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

Refactor GnssMeasurementsProvider

Bug: 77974102
Change-Id: I9d993bd7a4a32ab2eef293c0efae3109348e7577
Fixes: 77974102
Test: m -j ROBOTEST_FILTER=GnssMeasurementsProviderTest RunFrameworksServicesRoboTests

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

Work on issue #78480444: Start tracking uid state in app ops

We now push uid states down from activity manager in to app
ops, and it uses them to keep track of access/reject times for
each uid/package at the various states.

Bug: 78480444
Test: manual
Change-Id: Ia0bc9174b60b4bf0851834961cc48507a6a60951

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