Convert data/etc/Android.mk to blueprint

Should produce identical artifacts.

Test: boot blueline
Change-Id: Iad06f3efc727d97bdef3d4a7cec9b2b1fbc775b9

Merge "Add system_notification_accent_color for mainline"

Merge "Fix navigation bar flaky tests"

Merge "Clean up job concurrency calculation"

Merge "Move SystemUI and its permissions to /product"

Merge "Fix an issue in SurfaceTexture with forgetting the dataspace"

Merge "OMS: extract verifyIdmap from createIdmap"

Merge "OMS: fix checkstyle warnings"

Merge "idmap2: replace std::pair<bool, T> with Result<T>"

Merge "Move Settings privapp-permissions to /product"

Merge "[Multi-user] Decouple UserBMS and BMS creation"

Fix navigation bar flaky tests

The flakiness is because the timing issue on hasNavigationBar.
When a new display is added, onDisplayAdded is called and we may try to
add nav bar before DisplayPolicy#configure is called, which sets
mHasNavigationBar value. Therefore, we may get false from
hasNavigationBar even if a display does support system decorations.

Fixes: 120748674
Test: atest ActivityManagerMultiDisplayTests
Test: atest
ActivityManagerMultiDisplayTests#testNavBarShowingOnDisplayWithDecor *
100

Change-Id: Idb3c3d0874c15a466d179cc471ade5cea890e20e

Merge "Add metric fields for Smart Suggestions in Notifications logging."

Merge "Apply a per-package limit to secondary dex tracking."

Move SystemUI and its permissions to /product

Since this app is frequently customized by OEMs, it doesn't
fit in the mainline /system image right now. Move it to
/product.

Bug: 110072687
Test: make
Test: boot blueline
Test: atest SystemUITests
Change-Id: If27bcccd62f239f716db5ee7f4f1418d936e1cf4

Move Settings privapp-permissions to /product

The app is moving to /product, so move the permission
grants as well.

Bug: 110072687
Test: boot blueline
Change-Id: I34fcab908db5b7a96a50b45bb45941282b05a82d

Merge "Retrieve apps with recent location access"

Merge "Remove unused SystemUi service"

Remove unused SystemUi service

Bug:120864177

Test: emulator
Change-Id: I7fbd818b1210f2de3d89f3c2e35fd5dcad39d657

Merge "Replace InterfaceParams with ifName in DhcpServer" am: 9d5e00321f am: fce422d811
am: 5f5bbffb14

Change-Id: Ib41a931d97610ddeac60012560554c6642186b78

Merge "Replace InterfaceParams with ifName in DhcpServer" am: 9d5e00321f
am: fce422d811

Change-Id: Iaf8dd7a21156aba248d33d095c169c0f62d8eebe

Merge "Check that mDrivingStateManager is not null."

Merge "Merge "Add new netd event callback for changes to NAT64 prefix" am: ed30ec8dd1 am: dba2101a3c am: 97de461dcb"

Merge "Add new netd event callback for changes to NAT64 prefix" am: ed30ec8dd1 am: dba2101a3c
am: 97de461dcb

Change-Id: Ib08d3c201a71876a057176e2fce065e8360008e2

Merge "Fix build for libmedia2_jni"

Merge "Support wallpaper on secondary displays.(6/N)"

Merge "Add new netd event callback for changes to NAT64 prefix" am: ed30ec8dd1
am: dba2101a3c

Change-Id: I4e263d78c48a07fbcbfddc63e118c45b4cfb9c5a

Merge "Replace InterfaceParams with ifName in DhcpServer"
am: 9d5e00321f

Change-Id: Id2863d3c148b109c74ed37a0c2be7a4ddfd4f923

Merge "Support wallpaper on secondary displays.(5/N)"

Merge "About phone page getFactoryMacAddress"

Merge "Replace InterfaceParams with ifName in DhcpServer"

Merge "Add new netd event callback for changes to NAT64 prefix"
am: ed30ec8dd1

Change-Id: I5220971458e1491ccde1d8eeef0f91a9534aec75

Merge "Remove a special test-only mode from IMMS"

Merge "Add new netd event callback for changes to NAT64 prefix"

Merge "Add feature flag for settings slice injection"

Merge "Revert "Support mouse pointer on external displays (2/3)""

Merge "MediaPlayer2: add DataSourceDesc for source dependent API's"

Merge "Move biometrics into biometric namespace"

Merge "Sending a TIME_TICK on screen_on"

Merge "Prototype color changing support"

Merge "Scan Optimization: expose API to update device mobility state"

Merge "Remove unusued code; MediaPlayerBase and MP.set|getBufferingParams"

Fix build for libmedia2_jni

Mpeg2ts static lib now uses the hidl allocator
to get the hidl memory.

bug: 112766913
bug: 111407253
Change-Id: I588e9a603a0dd34dd90a69a32a9b355222507ff2

Merge "Add util for reverting schedule mode"

Merge "Reduce UsageStatsXmlV1 log messages."

Merge "Clean up some CarSystemUI theme elements"

Merge "Add multiple esim support for SubscriptionManager"

Merge "FileUtils.java: Don't treat open access modes as flags"

Merge "Add missing getter & improve docs"

Clean up job concurrency calculation

- Mostly variable renaming and comments.
- A few bugs have been fixed.
- I *think* the max job count check is not working as intended, but we don't
fix it yet.

Bug: 111360323
Test: atest CtsJobSchedulerTestCases

Summary
-------
CtsJobSchedulerTestCases: Passed: 33, Failed: 5

5 tests failed
--------------
android.jobscheduler.cts.BatteryConstraintTest#testBatteryNotLowConstraintFails_withoutPower
android.jobscheduler.cts.ConnectivityConstraintTest#testConnectivityConstraintExecutes_metered
android.jobscheduler.cts.ConnectivityConstraintTest#testConnectivityConstraintExecutes_withMobile
android.jobscheduler.cts.ConnectivityConstraintTest#testUnmeteredConstraintFails_withMobile
android.jobscheduler.cts.TimingConstraintsTest#testJobParameters_unexpiredDeadline

(Unchanged from the ToT result)

Change-Id: I9f0fe8c18265652ca2b6217b3f6540aa7233b7c0

idmap2: replace std::pair<bool, T> with Result<T>

Introduce a new type Result<T> to indicate if an operation succeeded or
not, and if it did, to hold the return value of the operation. This is
the same as how std::pair<bool, T> is already used in the codebase, so
replace all instances with Result<T> to improve clarity.

Result<T> is simply an alias for std::optional<T>. The difference is
semantic: use Result<T> as the return value for functions that can fail,
use std::optional<T> when values are truly optional. This is modelled
after Rust's std::result and std::option.

A future change may graduate Result<T> to a proper class which can hold
additional details on why an operation failed, such as a string or an
error code. As a special case, continue to use std::unique_ptr<T>
instead of Result<std::unique_ptr<T>> for now: the latter would increase
code complexity without added benefit.

Test: make idmap2_tests
Change-Id: I2a8355107ed2b6485409e5e655a84cf1e20b9911

Merge "Fix issue #121104681: Force stopping AGSA causes system to revert to its default impl"

Merge "Match current colors for device default car theme"

OMS: fix checkstyle warnings

Fix checkstyle warnings related to the overlay manager, i.e. in the Java
files under

  - core/java/android/content/om
  - core/tests/overlaytests
  - services/core/java/com/android/server/om

Test: prebuilts/checkstyle/checkstyle.py -f $path
Change-Id: I5078629240da2ceb1fd710d2e79e3114bafdd5bb

OMS: extract verifyIdmap from createIdmap

For clarity, split IIdmap2::createIdmap into two separate functions:

  - IIdmap2::verifyIdmap [check if an existing idmap file is OK to use]
  - IIdmap2::createIdmap [unconditionally (re)create an idmap file]

Teach the IdmapManager to call verifyIdmap and to proceed with
createIdmap only if actually needed.

Test: atest OverlayDeviceTests OverlayHostTests
Change-Id: I9f6f1192011fcb094adffeca1eb3f709520bbd24

Merge "Change platform Scene.getCurrentScene parameter from View to ViewGroup"

Merge "Statsd atoms: Add comment about Wifi"

Fix an issue in SurfaceTexture with forgetting the dataspace

Test: Ran CTS and verified dataspace is not set to 0
Bug: 119504473
Change-Id: I6f9920e1979e5435dc95a7c2ab3f02d57dc57c4e

Merge "Add TEST_MAPPING for androidfw to add presubmit tests."

Merge "Merge "Fix performance-for-range-copy warnings" am: bc29242288 am: 7adb73f332 am: 2296036a40"

Merge "Fix performance-for-range-copy warnings" am: bc29242288 am: 7adb73f332
am: 2296036a40

Change-Id: Ia99ea6c9633986e68340bc88983d06baa7a1c597

Merge "Initial sleep-lock-screen implementation"

Merge "Read product property to set Vulkan as default render pipeline"

Merge "Fix performance-for-range-copy warnings" am: bc29242288
am: 7adb73f332

Change-Id: I0ab1a28f5490f96bc8b08f073d364a01ff2f20df

Merge "Use frameworks wifi icons instead of sysui"

Merge "Fix performance-for-range-copy warnings"
am: bc29242288

Change-Id: I96e7ca53f2878fd9f75ac14999a02a96a96bb93e

Merge "[view-compiler] Add layout validation" am: ceea877ddd am: 1e41ae605d
am: 9020a7cd30

Change-Id: I62894502aa797466212ef3f91285ea0c5099f390

[Multi-user] Decouple UserBMS and BMS creation

The BMS constructor is now only responsible for setting up the system
service. User registration logic is moved to its own helper that
Trampoline calls during a user unlock callback.

** Logic **
All IBackupManager methods that BMS implements now use bookkeeping to
get the UserBMS instance to act on. Currently the system user instance
is hardcoded in most methods as we don't pass in the user id parameter
(except for a few added in ag/5667585).

If these methods are called on a non-registered user, we log and return
a default value.

** Tests **
Each IBackupManager method now has four corresponding tests:
1) Permission denial test: No INTERACT_ACROSS_USERS_FULL permission +
non-calling user id = security exception (added in ag/5667585, only for
methods that take in an user id param currently).
2) Permission grant test: INTERACT_ACROSS_USERS_FULL permission +
non-calling user id = call forwarded.
3) Registered user test: Passing an user id that has a corresponding
UserBMS instance -> delegate call to that UserBMS instance.
4) Unknown user test: Passing an user id that has no corresponding
UserBMS instance -> no call forwarded.

These tests will be updated when more methods take in an user id param.

Bug: 120212806
Test: 1) atest RunFrameworksServicesRoboTests
2) Unlock system user -> starts service, registers transports
3) adb shell bmgr flows
4) atest TrampolineTest
5) CTS + GTS Backup test cases
6) SUW + Cloud restore; D2D
Change-Id: Ic04f754c75af905ee34c609063c08406e23671d5

Merge "[automerger skipped] Merge "expose Carrier Id related APIs" am: 18b925a7af am: 0031ae8145 am: 31cf647396  -s ours"

Merge "[view-compiler] Add layout validation" am: ceea877ddd
am: 1e41ae605d

Change-Id: I2bfe50dbb03c6cdcb7cde9b1c1a8f2cf932c41e0

[automerger skipped] Merge "expose Carrier Id related APIs" am: 18b925a7af am: 0031ae8145
am: 31cf647396  -s ours

Change-Id: Id7d9376286fa9152532c9d967563d172e8601e8f

Merge "Fix performance-for-range-copy warnings"

Merge "Do not use default to get TelephonyManager"

Revert "Support mouse pointer on external displays (2/3)"

This reverts commit 84cdf9cc3b849679e6b4f6d3ccb66270f0abad2f.

Reason for revert: b/120864177

Test: Presubmit
Bug: b/120864177

Change-Id: Ia65e704169527b54cf1d1d470445f96a65fa0294

Merge "Changed verbosity of log message shown when context is not autofillable."

Merge "Implement ModuleInfoProvider"

Add feature flag for settings slice injection

Bug: 120803703
Test: Manual
Change-Id: I1e0f92576ffb01d7300ec60d9c40165454b3aeba

Merge "Fix checking if process name is whitelisted"

Merge "expose Carrier Id related APIs" am: 18b925a7af
am: 0031ae8145

Change-Id: I6e9837cffdcac094557ea99cde0736abdfff0ff9

Merge "[view-compiler] Add layout validation"
am: ceea877ddd

Change-Id: Ia95ec8ccc77c4be60f018d99391067f3b8b02fd2

Remove unusued code; MediaPlayerBase and MP.set|getBufferingParams

Test: build
Bug: 112766913
Change-Id: I999a637a1cb47a28a032478f3c56ab0645837f92

Merge "[view-compiler] Add layout validation"

Merge "Fix a race condition in DeviceIdleControllerTest."

FileUtils.java: Don't treat open access modes as flags

O_RDONLY, O_WRONLY, and O_RDWR are not flags. Rather, they are the
integer values 0, 1, and 2, respectively.

  #define O_RDONLY 00000000
  #define O_WRONLY 00000001
  #define O_RDWR 00000002

Quoting "man 2 open"

  * File access mode *

  Unlike  the  other  values  that  can  be  specified in flags,
  the access mode values O_RDONLY, O_WRONLY, and O_RDWR do not
  specify individual bits.  Rather, they define the low order
  two bits of flags, and are defined respectively as 0, 1, and
  2. In other words, the combination O_RDONLY | O_WRONLY is a
  logical error, and certainly does not have the same meaning
  as O_RDWR.

  Linux reserves the special, nonstandard access mode 3
  (binary 11) in flags to mean: check for read and write
  permission on the file and return a file descriptor that
  can't be used for reading or writing. This nonstandard access
  mode is used by some Linux drivers to return a file
  descriptor that is to be used only for device-specific
  ioctl(2) operations.

Rather than treat these values like flags, use O_ACCMODE to extract the
values and then perform the comparisons.

Introduced in 63280e06fc64672ab36d14f852b13df2274cc328.

Test: android compiles and boots.
Change-Id: I4d3185e835615ffba3a7854d3d58351e124599d0

Merge "expose Carrier Id related APIs"
am: 18b925a7af

Change-Id: I517cd9638544ced0117a7222b21c92940b52885e

Prototype color changing support

If enabled through settings (off by default) you can turn on color
adaption prototype that would sample colors around the home button to
determine if the button should be white or black.

Bug: 112934365
Test: manual
Change-Id: I726a005d97fd525c6b2d6ba3db9a8ed0cc355272

Merge "Deprecate ConditionProviderService."

Merge "passpoint-r2: define getMatchingPasspointConfigsForOsuProviders API"

Add TEST_MAPPING for androidfw to add presubmit tests.

Test: atest --host libandroidfw_tests
Change-Id: Id893e720139791b499f2f38a25a057738a488c2b

Merge "expose Carrier Id related APIs"

Merge "[Multi-user] Add -user param to adb backup/restore"

Statsd atoms: Add comment about Wifi

Add comment explaining caveats about the Wifi atoms.

Bug: 115639456
Test: It is just a comment.
Change-Id: I0882ccbe21c59439c4a6a61fb665bc0bf4695006

Merge "Statsd atom: Wifi on (enabled) and running"

Use frameworks wifi icons instead of sysui

ic_qs_wifi_full_x have been replaced by ic_wifi_signal_x in frameworks.
That way they can be overlaid with the whole system. Also,
ic_qs_wifi_disabled was identical to ic_qs_wifi_full_0, so it was
removed.

Test: Visual (using HotDog overlay)
Bug: 121040363
Change-Id: Ia36613e5c0b1f3a0c1eb5c687d795992e8660955

Fix performance-for-range-copy warnings

Bug: 30413223
Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,performance*
Change-Id: Ie481e88025a7a1f3abde8ff63420d5ccd8577e52

Fix issue #121104681: Force stopping AGSA causes system to revert to its default impl

If you have a legacy assist app (launched through ACTION_ASSIST, not
implementing a VoiceInteractionService) then we kept the Google app
as the current speech recognizer.  Thus if you force stopped the
Google app, we saw that as one of the active parts of the interactor
or recognizer and reset all of the state.

Now we handle the "only recognizer" case separately, only resettting
the recognizer state in that case.

Bug: 121104681
Test: manual
Change-Id: Icc007bdf2352548d58be997fae77d9e5aba842f3

[automerger skipped] Merge "Add FrequencyRange comparator to ServiceState" am: d0c404cb21 am: 5cb84d7f23
am: b527f8a44c  -s ours

Change-Id: I384994f412f4c6a79cac99387e42c3c2e113c3e2

Fix a race condition in DeviceIdleControllerTest.

I noticed that the real ACTION_BATTERY_CHANGED races against the mocked
calls to update power state and cycle through idle states. This actually
works on a high-end phone or an emulator because the sticky broadcast
goes through before the test finishes setting up.

But on a slower device we sometimes beat the broadcast queue and wind up
getting our state reset halfway through the test. Flaky tests are bad so
let's not listen to broadcasts at all in the test.

May address issue 118639768 [DeviceIdleControllerTest is flaky].

Bug: 110756616
Bug: 118639768
Test: atest com.android.server.DeviceIdleControllerTest
Change-Id: Ie03ed10f2781ccaba4f39f4f34673a3e073e9f7f

Merge "Add FrequencyRange comparator to ServiceState" am: d0c404cb21
am: 5cb84d7f23

Change-Id: I5550790e9db3604bf35d4f01b6e7fef316e3809e

Add missing getter & improve docs

Also fix SkiaDisplayList::getUsedSize()

Bug: 120865963
Test: atest android.uirendering.cts.testclasses.RenderNodeTests
Change-Id: I3cdbd8d330da9a00367199087c9ae1e0a2cacf46