Merge "MediaSession2: Clean up TODOs under frameworks/base" into pi-dev

Merge "pm: Modify getPackage() to only return non-null packages" into pi-dev

Disable LTO temporarily for hwui

Bug: http://b/62839002

LTO does not seem to interact well with PGO profile-file updates and
incremental builds in the build servers.

Test: build
Change-Id: I42175b2376b374d00ec169d4fe8d84cd938b21e5

MediaSession2: Clean up TODOs under frameworks/base

This change includes
  - Addressed trivial TODOs (takes less than 1min)
  - Renamed playlist controller to playlist agent
  - Renamed mplc to playlistAgent
  - Renamed mpb to player
  - Assigned buganizer issues for TODOs

Bug: 74361045
Test: Run MediaComponentsTest
Change-Id: I45e97081b5a3f24988c3daadd007edcb51be5288

Merge "Statsd cpu optimizaton" into pi-dev

Merge "Show app ops guts when tap app ops icons" into pi-dev

Merge "Tapping overview button when quickstep enabled shows overview" into pi-dev

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

Merge "Dumping stack traces to proto." into pi-dev

Merge "AudioFormat: java-native conversion for xHE-AAC" into pi-dev

Merge "Fix accessibility action parceling" into pi-dev

Tapping overview button when quickstep enabled shows overview

This is the systemui side of showing launcher's overview when it decides
to allow users to trigger overview with the overview button in the
navigation bar when quickstep is enabled.

Test: manual - tap overview button in nav bar
Bug: 67957962
Change-Id: I0d59d60d4b4ce1df9dcf2a745f9a5efd415b8582

Show app ops guts when tap app ops icons

Test: runtest systemui
Bug: 64085448
Change-Id: Ia51044f77311ab705ffb9dd1ae204d0131316396

Merge "Do not depend on task activities decrementing with finish." into pi-dev

Statsd cpu optimizaton

- cache the changed dimensions in condition tracker.
- avoid query condition wizard when unnecessary.
- avoid copy dimension keys in condition key generation.

Test: statsd tests.

BUG: b/73959649
Change-Id: I17d68e2a82643de3f421309841e75f84c6fd8f43

Merge "Add colorProgressBackgroundNormal" into pi-dev

Merge "Implement front-end support for HAL 2.0 metadata." into pi-dev

Import translations. DO NOT MERGE

BUG:72647570

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

Merge "AudioFormat: Uncomment conversion for EAC3-JOC" into pi-dev

Merge "Refactor cpu stats pullers" into pi-dev

Merge "Show system uptime in bugreport (dumpsys alarm)" into pi-dev

Merge "Fix NPE if app context is null" into pi-dev

Merge "Verify the content length in the verity digest" into pi-dev

Merge "Don't consume all BACK keys in SearchView" into pi-dev

Merge "Setting the alpha on the full bitmap for AdaptiveIcon instead of individual layers" into pi-dev

Merge "Fix mismatch in parcel read/write in ParcelableCasData -- DO NOT MERGE" into oc-dev am: 5d2153d17c am: 2d0e5b6acc  -s ours
am: c551f11cd6  -s ours

Change-Id: Id5ade2438e3a4716ca4ac69fa11438169dd2e763

Merge "Don't store lockout deadline in lockSettings" into pi-dev

Merge "Fix mismatch in parcel read/write in ParcelableCasData -- DO NOT MERGE" into oc-dev am: 5d2153d17c
am: 2d0e5b6acc  -s ours

Change-Id: I231859f9e26c99ae3a34241842e50f4d44d23f93

Merge "Fix mismatch in parcel read/write in ParcelableCasData -- DO NOT MERGE" into oc-dev
am: 5d2153d17c

Change-Id: I04e026a96e1899ba9496979561df5680297f546a

AudioFormat: java-native conversion for xHE-AAC

Bug: 71430241
Test: only once we have decoder
Change-Id: Ifab4e887d1516da45eed762a8a4d92c3b8f1f95c

Merge "Fix mismatch in parcel read/write in ParcelableCasData -- DO NOT MERGE" into oc-dev

Merge "Window: Provide frame size hint to ViewRootImpl" into pi-dev

Merge "Apply prior tint to rotate icon on recreation" into pi-dev

Apply prior tint to rotate icon on recreation

In very rare circumstances, it's possible for the rotate icon
to be recreated but not have the dark intensity set. Fix this
by applying prior icon's intensity.

Test: manual but heisenbug so very hard to validate
Fixes: 74354524
Change-Id: I042de35e725794b04f5250fa3a2fc5a54d65d143

Add colorProgressBackgroundNormal

Adds new private theme attr colorProgressBackgroundNormal which
the default material/devicedefault progress/seekbar backgrounds
use for their background tint.

Added default values for Material/DeviceDefault themes. Material's
is colorControlNormal.

Adds a new config_progress_background_tint color resource that OEMs can
overlay. Default value is colorControlNormal since AOSP should not
change the tint. OEMs who wish to change the default can simply
overlay config_progress_background_tint with their preferred color.

Finally, deletes the unnecessary control_nodisable_material which
was just a layer of indirection pointing to colorControlNormal.

Bug: 69314526
Test: ag/3665368
Change-Id: I6443e08469677cdf31d99c73966b437e5a1f2472

Merge "Improve a11y pane title javadoc" into pi-dev

Implement front-end support for HAL 2.0 metadata.

Bug: 73816100
Test: instrumentation
Change-Id: I1d4e9094f7a2eaf13a14f5fa560a391fcbae9804

Merge "Screenshots broken for activities with dialog themes" into pi-dev

Merge "Revert "Revert "pm: SharedUserId: Assign seinfo using actual targetSdkVersion""" into pi-dev

AudioFormat: Uncomment conversion for EAC3-JOC

The corresponding native constant has been added to system/audio.h

Bug: 38184704
Bug: 63901775
Test: make
Change-Id: I141904571755ddb67491e72c73938953034283a0

Merge "Introduced DISABLE_COMPATIBILITY_WAL flag" into pi-dev

Don't consume all BACK keys in SearchView

If SearchView is the first focusable, it will always get focus
(in non-touch-mode) when it tries to clearFocus on BACK pressed.
This lead to a situation where SearchView always consumed the
BACK key leaving users unable to leave some activities.

It looks like this was done so that pressing back both closed the
auto-correct popup AND the ime (whereas without reimplementing
onPreIme, it would require 1 back-press to close the auto-correct
popup and then a subsequenty press to close the IME). It also,
however, tried to clearFocus as well.

This change only consumes the Back press if the auto-correct popup
is open (to have the same effect of BACK closing both the popup
and the IME at the same time). It ignores the back press otherwise.
If there is no pop-up, this results in the BACK being handled by
the ime and thus hiding it. Otherwise, back is not consumed.

The only effective difference is that the SearchView remains
focused now.

Bug: 73181998
Test: SearchView CTS tests still pass. Back can now exit test-app
      once IME is hidden.

Change-Id: I3fe687b5344300b86131f44a1c9798cd736955bd

Dumping stack traces to proto.

Bug: 72177715
Test: flash device and check incident.proto output
Change-Id: Id2a15e0fc62b66efe875949af97f0eb651c7e322
(cherry picked from commit 5c804e2b9893c7892900148229cf19fa6268e7dc)

Refactor cpu stats pullers

use same cpu stats readers with BatteryStats so that
1) both throttle to avoid too frequent pulls
2) cached value is served within throttle interval to avoid double
pulling by both statsd and BatteryStats

To run unit tests:
bit
FrameworksCoreTests:com.android.internal.os.KernelUidCpuFreqTimeReaderTest
bit
FrameworksCoreTests:com.android.internal.os.KernelUidCpuClusterTimeReaderTest
bit
FrameworksCoreTests:com.android.internal.os.KernelUidCpuActiveTimeReaderTest

make -j56 statsd_test && adb sync data && adb shell
/data/nativetest64/statsd_test/statsd_test

Test: cts test, unit test
Bug: 73745189
Bug: 73780619
Bug: 73360959

Merged-In: I10a9bc91ca67fa812f4cd71c4fbd73c1a5ba580e

Change-Id: I10a9bc91ca67fa812f4cd71c4fbd73c1a5ba580e

Merge "Introduce new perf test for TextView with precomputed text" into pi-dev

Merge "AudioService: don't send BECOMING_NOISY when dyn policy affects media" into pi-dev

Merge "Fix separator color for light wallpapers" into pi-dev

Verify the content length in the verity digest

When generating digest for verity, for the last incomplete 4k chunk, the
data is padded with 0s.  This implies that we can not tell from the
digest whether the file contains 0 or not, or how many 0s.

Since the verity hash is used by the kernel, the definition cannot be
change.  Instead, the actual hashed content length is appended to the
original digest and is verified before used.

Also uprev algorithm IDs.

Test: use new apksigner to sign an apk, apk can be installed on device
Bug: 30972906
Change-Id: I382af6e4090c7dc3f92d5acb5ac5d02d1f496992

Merge "DO NOT MERGE - Disable warning on blocking one-way call from autofill server." into pi-dev

Merge "Throw an exception in case of parameter mismatch of precomputed text" into pi-dev

Do not depend on task activities decrementing with finish.

It is possible for a task to be cleared when a single activity is
finished. For example, if only task overlays are present after an
activity is removed, the task can be cleared under certain
conditions. As a result, we can not rely on only a single activity
being removed from a task when it is finished.

This changelist addresses this issue by caching the list of actvities
under a task that will not be modified as activities are finished.

Change-Id: Id3b0813deebd0bc31b2ff7ae7f69a2833dcb0a61
Fixes: 64291682
Test: atest FrameworksServicesTests:com.android.server.am.ActivityStackTests#testFinishDisabledPackageActivities
Test: atest FrameworksServicesTests:com.android.server.am.ActivityStackTests#testHandleAppDied

Merge "Ignore events when there is no focused stack." into pi-dev

Merge "Some more members for the light greylist." into pi-dev

Merge "Implemented notifyViewClicked() on compat mode." into pi-dev

Fix separator color for light wallpapers

Test: visual
Change-Id: I61b2161db16db264c7425431dca50f5a54fce425
Fixes: 74345301

Introduce new perf test for TextView with precomputed text

Here is an example of perf test result:

TextView new layout creation time:
  PrecomputedText           :    740,173
  PrecomputedText_Selectable: 17,727,649
  RandomText                : 17,130,685

TextView onDraw time:
  PrecomputedText           :  1,471,075
  PrecomputedText_Selectable: 31,971,393
  RandomText                : 17,667,572

TextView onMeasure time:
  PrecomputedText           :    177,669
  PrecomputedText_Selectable: 17,773,204
  RandomText                :  6,296,358

TextView setText (w/o layout creation) time:
  PrecomputedText           :     91,858
  PrecomputedText_Selectable:    152,665
  RandomText                :     11,130

Bug: 72998298
Test: ran perf test
Change-Id: I69af74ba743499bc444c441c1472fb19878c2eec

DO NOT MERGE - Disable warning on blocking one-way call from autofill server.

This behavior existed since autofill was introduced on O and it won't be fixed
on P, so there's no point on warning. In fact, the warning is often a red
herring for other issues, not to mention a big logcat spammer.

Bug: 35708678
Test: manual verification looking at logcat

Change-Id: I40be4ce25abc5b097ea67e5cb34bb9c4237f0826

Merge changes from topics "pick-tdscdma", "wcdma-rscp" into pi-dev

* changes:
  Update SignalStrength to use ASU for TD-SDCMA
  Customize the WCDMA signal strength based on RSCP

Fix NPE if app context is null

Change-Id: Ic21ae50e5421a8c711e133ffe2ff218160382297
Fixes: 74260094
Test: builds, boots, and graphicsstats still works

Some more members for the light greylist.

Test: N/A

Bug: 74323509
Change-Id: I142b7f750470aa0cc49ff5f17866516014d615e6

Merge "Draw AnimatedImageDrawable mirrored if desired" into pi-dev

Revert "Revert "pm: SharedUserId: Assign seinfo using actual targetSdkVersion""

Commit 042f4bb6 'pm: Modify getPackage() to only return non-null
packages' prevents the NullPointerException. b/
This reverts commit 00b4fa8b138de801b0caf371a950fcda7b98bb83.

Bug: 74214574
Bug: 72290969
Test: Same tests run on original commit.
Test: successfully flash and boot from bild 4504078 to ToT master.
Change-Id: Ia1e900ebe0eb8f9d10fd0eddbf658f52e1ea79b4
(cherry picked from commit 32ef1c2eb0fe32d8fd02a44bc77b40ab7719c433)

Throw an exception in case of parameter mismatch of precomputed text

If the given precomputed text is not compatible with the TextView,
reject the text by throwing IllegalArgumentException.

Bug: 73091756
Test: atest CtsWidgetTestCases:EditTextTest
    CtsWidgetTestCases:TextViewFadingEdgeTest
    FrameworksCoreTests:TextViewFallbackLineSpacingTest
    FrameworksCoreTests:TextViewTest FrameworksCoreTests:TypefaceTest
    CtsGraphicsTestCases:TypefaceTest CtsWidgetTestCases:TextViewTest
    CtsTextTestCases FrameworksCoreTests:android.text
    CtsWidgetTestCases:TextViewPrecomputedTextTest

Change-Id: I4fbf89a5f1409e8eefdeb9f208f9a3758220fe1a
(cherry picked from commit 3a0787af5e1bb1f615c1dd2192a08464ea3776e3)

pm: Modify getPackage() to only return non-null packages

Bug: 74214574
Test: successfully flash and boot from build 4504078 to ToT master
Change-Id: I458db34a441dabed3cad7bd8fc9769bad49b3b30
(cherry picked from commit b6c042f4bb65cf322151e50a3090e4b1f6304b2a)

Setting the alpha on the full bitmap for AdaptiveIcon instead of individual layers

Test: atest FrameworksCoreTests:AdaptiveIconDrawableTest
Bug: 72798522
Change-Id: I3862942bf6f2c4767c293f09ad14725b5f0fbb2b

Merge "Avoid service wrapper initialization deadlock." into pi-dev

Improve a11y pane title javadoc

Bug: 74123987
Test: Just changing comments.
Change-Id: I7f21947a35d1baccf171346979164718386c5b71
(cherry picked from commit 50d2808b958b1568d71a7d42519f924b479f642d)

Screenshots broken for activities with dialog themes

Makes progress towards fixing screenshots and transitions
for non-fullscreen activities.

Partially reverts I48053f551ed1c118764aa30e0b6408db64d4a5f1 and
applies it in a way that does not break screenshots in general.

Note that screenshots of non-fullscreen activities are still somewhat
broken because their insets can currently not be calculated correctly.

Bug: 72757033
Test: Open dialog activity, go to recents, verify it has a reasonable screenshot and smooth transition
Test: Open letterboxed activity, go to recents, verify it has a reasonable screenshot and smooth transition
Change-Id: Ica4d88370df00a91724d59b2cad6b27826beba04

Merge "Fix duplicate text in battery string" into pi-dev

Merge "Remove BIND_SLICE" into pi-dev

Fix accessibility action parceling

We were using an int for a bitmask, but the number of
standard actions reached 32 with the tooltip actions.
Switching to long.

This CL is mostly ag/3662206, but that CL did not
convert everything to ints.

Bug: 73897184
Test: AccessibilityNodeInfo tests caught this bug. I've also
enhanced it to make sure that all standard actions work
across parceling.

Change-Id: I2adae3ec34ac327af354449fbe6511f01cc30b2f
(cherry picked from commit c9bf45564b58f85e998d83e4ac797350478bbba6)

Merge "Catch and log exceptions from setup or teardown of WindowTestBase" into pi-dev

Merge "Disable color extraction for media" into pi-dev

Merge "Don't use highlighting in non-selectable text. Also fixes potential discrepancy in indexes for Linkified entities." into pi-dev

Implemented notifyViewClicked() on compat mode.

Test: atest CtsAutoFillServiceTestCases:android.autofillservice.cts.VirtualContainerActivityCompatModeTest#testSave_submitButtonClicked

Fixes: 73649008

Change-Id: Icf6a8b617e06055c58e38c5b7208c7f3ef3121ee

Don't store lockout deadline in lockSettings

The deadline is enforced in gatekeeper, which persists after reboot.
Users should not see the lockout timer after rebooting the device if it
was previously in lockout, until an unlock attempt occurs.

Fixes: 66443036

Test: 1) fail pattern five times
2) reboot device
3) go to bouncer, lockout timer should not show
4) upon entering pattern, timeout should show (if done fast enough)

Change-Id: Ie07945b40c2f482fd1df7102ba96283ae9d98d75

Merge "Validate restore of NetworkPolicy settings" into pi-dev

Merge "System sounds get its own dnd toggle" into pi-dev

Merge "Clarify javadoc for DevicePolicyManager.setTrustAgentConfiguration" into pi-dev

Ignore events when there is no focused stack.

Skip brightness slider events when there is no
focused activity stack.

Bug: 73825714
Test: atest SystemUIGoogleTests
Test: atest BrightnessTrackerTest
Change-Id: I87376bbab47545dbcd9be684d1c8174fc9c8d92f

Merge "Remove com.android.internal{.util} from generated documentation." into pi-dev

Window: Provide frame size hint to ViewRootImpl

Provides a better estimate to the ViewRootImpl about the likely
window frame size than it can get from the configuration, which
allows it to avoid unneccessary measure passes.

Bug: 73813813
Test: atest PhoneWindowManagerLayoutTest
Change-Id: Ia4de6509416d90eed61929ce1646904c31afdd8d

Merge "Fix extra padding when there are no static notification dots" into pi-dev

Merge "Screenshots: delete, default for edit intent." into pi-dev

Merge "Removed @FlakyTest from ScreenDecorWindowTests" into pi-dev

Clarify javadoc for DevicePolicyManager.setTrustAgentConfiguration

Fixes: 36887130
Test: builds
Change-Id: Ieccd3edc0122c9ae49903a3c5fc9fb0aba6c08a2

Merge "Fix issue with where display is removed while creating it in AM and WM" into pi-dev

Remove com.android.internal{.util} from generated documentation.

com.android.internal.* is meant to be hidden from documentation,
but most of it is erroneously not hidden via @hide or -hidePackage
directives; why documentation is currently generated for Predicate
but not other classes from com.android.internal.util, and why some
but not all classes from that package show up in package-level
documentation (package-summary.html), is not currently understood.

There appears to be a behavior difference between OpenJDK 8 and
OpenJDK 9's javadoc that results in additional classes showing up
in package-summary.html. This CL fixes this by adding -hidePackage
directives for com.android.internal{.util}; other sub-packages of
com.android.internal do not currently show up in documentation and
are not touched by this CL.

Test: Patched this CL into the internal-master branch and ran:
      USE_R8=true EXPERIMENTAL_USE_OPENJDK9=true make offline-sdk-docs
      Checked that this removes all documentation for com.*
      (com.android.internal.util was the only com.* package for which
      documentation was previously generated).
      In other words: Before this CL, [1] existed, but after
      this CL, the entire directory subtree [2] does not exist.
Test: Checked that Predicate was already missing from stubs before this
      CL. In other words, [3] already did not exist before this CL.

[1] out/target/common/docs/offline-sdk/reference/com/android/internal/util/Predicate.html
[2] out/target/common/docs/offline-sdk/reference/com
[3] out/target/common/obj/JAVA_LIBRARIES/android_system_stubs_current_intermediates/classes/com

Bug: 69736344
Bug: 69736236

(cherry picked from commit 97bb6cf371f51549c708cf58a04584968eaf7d5a)

Merged-In: Ic9757f4966f54092aac0191896581fa4222cc634
Change-Id: Ic9757f4966f54092aac0191896581fa4222cc634

Merge "frameworks/base: Set LOCAL_SDK_VERSION where possible." into pi-dev

Merge "Block adb from changing certain settings value when corresponding user restriction is on." into pi-dev

Removed @FlakyTest from ScreenDecorWindowTests

Issues causing test to be flaky have been fixed.

Change-Id: Ie167c89bf32d24adb20766d2a36773b3f268d9ee
Fixes: 68957554
Test: this

frameworks/base: Set LOCAL_SDK_VERSION where possible.

This change sets LOCAL_SDK_VERSION for all packages where
this is possible without breaking the build, and
LOCAL_PRIVATE_PLATFORM_APIS := true otherwise.

Setting one of these two will be made required soon, and this
is a change in preparation for that. Not setting LOCAL_SDK_VERSION
makes the app implicitly depend on the bootclasspath, which is
often not required. This change effectively makes depending on
private apis opt-in rather than opt-out.

Test: make relevant packages
Bug: 73535841
Exempt-From-Owner-Approval: Global cleanup
Change-Id: I26458e41ecb84de91ac9a356a5d4bafb44f463c1

Merge "MediaSession2: Fix random NPE in MediaBrowser2Test" into pi-dev

Block adb from changing certain settings value when corresponding user
restriction is on.

Check calling uid in isSettingRestrictedForUser(which is called by settingsprovider),
 and only allow system_uid when certain user restriction is on, so that user won't be
able to change these settings with adb:
Settings.Secure.LOCATION_MODE,
Settings.Secure.PROVIDERS_ALLOWED,
Settings.System.SCREEN_BRIGHTNESS,
Settings.System.SCREEN_BRIGHTNESS_MODE,
Settings.System.SCREEN_OFF_TIMEOUT,
Settings.Global.AUTO_TIME,
Settings.Global.AUTO_TIME_ZONE.
This check also prevents 3rd party apps from modifying system settings value
when corresponding user restriction is on.
In addition, any attempt to change AUTO_TIME will also go through the check
for dpm.getAutoTimeRequired().

Test: manually by running the adb command with restriction set and not set
Bug: 72549013
Bug: 72548203
Bug: 72548533
Bug: 72686466
Bug: 72687105
Bug: 72940551
Bug: 72940562
Change-Id: I1d1fd20d9fa0f76f27905d62873f6a6e9af0224e

Merge changes from topics "mplc", "session2_onDataSourceMissingHelper" into pi-dev

* changes:
  Remove MediaPlaylistController, add MediaPlaylistAgent
  MediaSession2: Add SessionCallback#onCurrentMediaItemChanged()
  MediaSession2: Add onDataSourceMissingHelper

Remove MediaPlaylistController, add MediaPlaylistAgent

MediaPlaylistAgent is the abstract class an application needs to
derive from to pass an object to a MediaSession2 that will override
default playlist handling behaviors. It contains a set of notify*
methods to signal MediaSession2 that playlist-related state has
changed.

Bug: 64098437
Test: make update-api
Change-Id: Icb3c57ddc14eba276f49d4ba85f11adbeb3e0917

MediaSession2: Add SessionCallback#onCurrentMediaItemChanged()

Added callback for session to know the currently playing media item has
changed.

Note that the callback is called in response to the
MediaPlayerBase#PlayerEventCallback#onCurrentDataSourceChanged(mpb, dsd
is called. Session will translate dsd to the media item and calls
onCurrentMediaItemChanged().

Following changes are also included
  - Removed MediaPlaylistController#getCurrentPlaylistItem(),
    because currently playing item is managed by the MediaPlayerBase.
  - Renamed ControllerCallback#onCurrentPlaylistItemChanged() to the
    ControllerCallback#onCurrentMediaItemChanged(), to make it more
    obvious that the event is from MediaPlayerBase, not
    MediaPlaylistController.
  - Added SessionCallback#onCurrentMediaItemChanged()

Bug: 64098437
Test: Run MediaComponents test
Change-Id: I78b124a7da0f968b097b2576507b9a73e36081ec

MediaSession2: Add onDataSourceMissingHelper

This allows a developer to create DataSourceDesc when the item is about
to be played. Typical example of the usages are,

    1. For a playlist consists of FileDescriptors, its developer may not
        want to open all files when MediaSession2.setPlaylist() is
        called.
    2. A controller has called setPlaylist(), addPlaylistItem(), or
        replacePlaylistItem(). Controller cannot know the
        DataSourceDesc, and only the session developer can know about
        it.

Bug: 64098437
Test: Run MediaComponents test
Change-Id: I73f27ca0a799b1cddf5046b41f0ca01d08037103

Merge "Connectivity: Fix map access" into pi-dev

Disable color extraction for media

Overlaying an album with wallpaper colors isn't optimal.
Using the album extracted color also isn't optimal, the color probably
won't meet accessibility guidelines and will have to be stretched
according to the current lock screen theme - which can be even worse.

Test: atest packages/SystemUI/tests/src/com/android/systemui/colorextraction/SysuiColorExtractorTests.java
Change-Id: I53d08713716bd76ee0975c2b4bba5b933201f999