In particular
add events to cover
-If this is KV or Full restore
-For each package, when the restore started and when it ended
-When a restore operation starts, if it is system restore or restore at install
-Any errors

Test: manual testing. Run `adb shell bmgr restore 1` and verify that the
new restore events are added to the dumpsys
atest CtsBackupHostTestCases, GtsBackupHostTestCases
atest BackupManagerMonitorDumpsysUtilsTest,
BackupManagerMonitorEventSenderTest, UserBackupManagerServiceTest,
PerformUnifiedRestoreTaskTest, BmgrTest
Bug: 290747920

Change-Id: I0d221f10932fea3e8fb90a1827c7f1b5bf21d25d

Bug: 295394788
Test: m checkbuild
Change-Id: I6aa039b6b2a4944e3537ef133f8785890d957edd

Bug: b/175635923
Test: treehugger
Change-Id: I241c8b2b48bb4c616fbc5b4ae1285e274182144b
Merged-In: I241c8b2b48bb4c616fbc5b4ae1285e274182144b

Bug: 295394788
Test: make checkbuild
Change-Id: I95264e8bfc96a5adb7f7cc55a24dc0e8c126371d
(cherry picked from commit a550a478)

Bug: 295394788
Test: make checkbuild
Change-Id: I370f66c469de73064dec2e42c539dc236dd69d1e

Bug: 295394788
Test: make checkbuild
Change-Id: I95264e8bfc96a5adb7f7cc55a24dc0e8c126371d

Bug: b/175635923
Test: treehugger
Change-Id: I241c8b2b48bb4c616fbc5b4ae1285e274182144b

Don't convert each attribute name but instead only
do it once for the string they're compared to

Bug: 282215580
Test: build + atest idmap2_tests
Change-Id: I09f6d0f90f6c09632ceac8cdde7c20801f41524e

Bug: 295394788
Test: make checkbuild
Change-Id: I538720d4de2dcc038b417048209782ff605b7e30

This empty method is different from the old one - it aligns with
std::string definition.

Bug: 295394788
Test: make checkbuild
Change-Id: I1900997f36327af8c3d12b4f77971fdd96080616

This empty method is different from the old one - it aligns with
std::string definition.

Bug: 295394788
Test: make checkbuild
Change-Id: Ib80c839df7ff7e3fb0ed2df7a00393d843d44d9d

Bug: 295394788
Test: TH
Change-Id: I8e87ed078797cc941a16d1b2a3f96e0d668bf4ec

Bug: 295394788
Test: m checkbuild
Change-Id: I2e3167aaf464132127d5838075c7f9a5257ed275

See https://source.android.com/setup/contribute/respectful-code for reference

For this round, the fixes are only applied to the following to minimize breaking dependencies:
  * comments (excluding javaDoc annotations)
	* private constants
	* private functions
	* parameters within functions

BYPASS_INCLUSIVE_LANGUAGE_REASON=Just updating a few select inclusive language violations.
No-Typo-Check: Changes focused on inclusive language violations.
BUG: 295342157
Change-Id: Iacc0ea9fc848b15048978637bdc25136582433ef

Bug: 295394788
Test: make checkbuild
Change-Id: I82d6899d8c15a10b15399c39177290012bb5f13b
Merged-In: I82d6899d8c15a10b15399c39177290012bb5f13b

Bug: 295394788
Test: make checkbuild
Change-Id: I82d6899d8c15a10b15399c39177290012bb5f13b

Bug: 291881033
Bug: 294275314
Test: manual
Change-Id: Ie522ff44aa1cb7ceb7f21a23d6dd68efd1845f7c
Merged-In: Ie522ff44aa1cb7ceb7f21a23d6dd68efd1845f7c

Bug: 291881033
Bug: 294275314
Test: manual, presubmit
Change-Id: I928da2ad53afa91bc9fe7b0dc9cddeb3c918c6a2
Merged-In: I928da2ad53afa91bc9fe7b0dc9cddeb3c918c6a2

Bug: 294275314
Test: manual
Change-Id: Ie522ff44aa1cb7ceb7f21a23d6dd68efd1845f7c

Bug: 294275314
Test: manual, presubmit
Change-Id: I928da2ad53afa91bc9fe7b0dc9cddeb3c918c6a2

Before this change, if a (system) app tried to use an image from its
assets to create an FRRO it would succeed but no image would display.
That is because when it is an asset we have an AssetFileDescriptor which
internally has a file descriptor, offset, and size and the file
descriptor would point at the apk itself. When this would get passed to
the native layer to create the frro only the file descriptor would get
passed so we would copy the whole apk, not just the image.

This allows apps to pass in an AssetFileDescriptor directly which we can
then use to also pass down the offset and size to the native layer and
only copy the image part.

Fixes: 289739832
Test: atest FabricatedOverlayTest
Change-Id: I80dceb19229a6a3c02851a5451ea6d46798981b3

This greatly improves the performance of locksettings commands, and it
eliminates the need for LockSettingsCmd.java.

Bug: 177985929
Test: locksettings command still works
Change-Id: I5881fa074681f6bf1ad2f5969a282d928121ddb2
Merged-In: I5881fa074681f6bf1ad2f5969a282d928121ddb2
(cherry picked from commit 4d38b0cd)

Updating check for captureDisplay ID error status.

Bug: 287081404
Test: atest displayScreenshot
Test: atest ScreenCaptureTest#captureDisplayWithUid
Change-Id: Ic1cdf47a5e5044ac25ab6b7e420d29b8421c8b5d

Using the evedev event label strings directly in the uinput device's
command will make the commands much more readable and developer
friendly.

Bug: 293327283
Test: atest CtsInputTestsCases
Change-Id: I56ac7b228613ddffae3fec5261885de54db78c2c

pushedEventThread references class members after detaching. Making
pushedEventThread as class member and joining in statsService
destructor. Adding a method to stop readLogs thread.

Ignore-AOSP-First: Bug is in still security triage and fuzzer is
crashing on startup.
Test: atest statsd_test
Test: m statsd_service_fuzzer && adb sync data && adb shell
/data/fuzz/arm64/statsd_service_fuzzer/statsd_service_fuzzer -runs=10000
Bug: 285645039

Change-Id: I1e886f9ccb7203714216da061c35e793b2a63d8a
Merged-In: I1e886f9ccb7203714216da061c35e793b2a63d8a

pushedEventThread references class members after detaching. Making
pushedEventThread as class member and joining in statsService
destructor. Adding a method to stop readLogs thread.

Ignore-AOSP-First: Bug is in still security triage and fuzzer is
crashing on startup.
Test: atest statsd_test
Test: m statsd_service_fuzzer && adb sync data && adb shell
/data/fuzz/arm64/statsd_service_fuzzer/statsd_service_fuzzer -runs=10000
Bug: 285645039

Change-Id: I1e886f9ccb7203714216da061c35e793b2a63d8a

This change moves the bootanim time file location from
/data/bootanim/time to /data/misc/bootanim/time. This is to address the
recommendation in b/232299581#comment8.

Bug: 210757252
Test: Boot animation can show time. No SELinux violations.
Change-Id: I8fcf9943911f6f3f0af4d6c3ae391c5805bf7885

The main changes are:

* Describe how multiple commands are passed to `uinput`
* Split details about waiting after device registration into its own
  section
* Add details on the JSON extensions `uinput` uses
* Change example code blocks to JSON5 syntax to better accommodate the
  JSON extensions
* Use absolute links to Android Codesearch instead of relative ones
* Fix various Markdown issues and improve formatting

Bug: 280604085
Test: check rendering in Google Markdown
Change-Id: Ic6d5f3e9d1415fcd9425b1afc767574a65556578

This CL is a postmortem action which is detailed in the linked bug.
In short, an OEM had the telecom stack outside of the system process
and had a non-ui InCallService it wanted to bind.  However, the
InCallService set the Exported property to false.  This combo caused
a calling outage that prevented users from making/receiving calls.

To prevent the above scenaro from occuring again, this new CTS test
was created. It tests the Telecom stack of the OEM against a
non-ui InCallService that has it's exported property set to false.

Bug: 198715680

Test: 1 new CTS test,
      NonUiInCallServiceWoExportTest
      #testTelecomCanBindToNonUiInCallServiceThatIsNotExported

Change-Id: I5b99aa9c7e989a1c20a7b086f65782286a8be575

BUG: 288145890, 288146349, 288138476
Test: presubmit
Change-Id: Ie20f7d992d456db8e3f8e8b69297bcbd91bf6ead

This reverts commit cf6e79f8

Updates:
    Prepare the shared pointers for the whole operation at once
    instead of re-locking them on each iteration.
    Still a regression of about 5% for changing theme's
    AssetManager object, vs the original 40%

    + change the log message to a warning as it doesn't break
      the app

Original comment:

    Use reference counted pointers for ApkAssets

    The primary reason for memory corruption is freed ApkAssets
    Java expected them to only be freed in the finalizers, but
    there are explicit close() calls now, destroying objects that
    are still in use in some AssetManager2 objects

    This CL makes sure those AssetManagers don't assume ApkAssets
    always exist, but instead tries to lock them in memory for any
    access

    It also adds logging in case of deleting an assets object with
    any weak pointers still existing. Those will get into the
    bugreports attached to related bugs to help with investigation.

    Benchmarks don't regress, and the device appears to be working.
    Given that the crashes used to be pretty rare, let's wait for
    any new reports or lack of those.

    + add a missing .clang-format file to the jni directory
    + enabled C++23 in the project that uses AssetManager headers

    Bug: 197260547
    Bug: 276922628
    Test: unit tests + boot + benchmarks
    Old change id: I495fd9e012fe370a1f725dbb0265b4ee1be8d805
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:b3455190124b41e2deb1774d9c05b396b73b41a2)
Merged-In: Id668fbcf07db17b09691a344c04e98df83006f97
Change-Id: Id668fbcf07db17b09691a344c04e98df83006f97

The underlying UiAutomation object uses classes which expect
the main looper to have been prepared.

Fix: 289444999
Test: adb shell uiautomator dump
Test: Use go/hsv to take a snapshot
Change-Id: I43fcc4e9b4c1ab6e09aacd73f638208094641554

The variable "mapbase" is not really used and we should remove it from screencap

Test: make screencap

Change-Id: Id17b6f440d3ab42f1c54fe90065eac50e0190f01
Signed-off-by: viktor <tomjhin1@gmail.com>

When bootanimation contains part that is played only once,
there is no need to create a separate texture for its frames.

Such an approach allows to reduce memory consumption for
typical use case of such an animations, when a part
consists of a lot of frames played only once.

Actually, it was already implemented earlier, but part of it
was lost in 'd711ac48'.
Moreover, the remained part leads to memory leak in
current implementation on BootAnimation.cpp:1712.

Test: play bootanimation with part.count=1 and a lot of
frames, and check memory consumption.

Change-Id: If3a0fa890f6f9e7abc0db2fe96b5635a1663fb8a
Signed-off-by: Oleg Lyovin <ovlevin@sberdevices.ru>

readFileDescriptor doesn't provide ownership of the fds. fdopen
needs ownership of the fds. Fds read from parcel should be duped
in this scenario and readUniqueFileDescriptor dups fds internally.

Test: m incidentd_service_fuzzer && adb sync data && adb shell /data/fuzz/x86_64/incidentd_service_fuzzer/incidentd_service_fuzzer
Test: atest incidentd_test
Bug: 286931110
Bug: 283699145
Change-Id: Ibe03a17dee91ac5bf25d123d4fd9c0bdd3c7d80e

Ibe3679d810602456205e157fe2cb8cc510062215 restricted the argument to
PhysicalDisplayId, which prevents capture of virtual displays.

Fixes: 280785757
Test: screencap -d <id>
Change-Id: I41a8d1adaa51c6804c03f1b3084ac04ff6b4f1fa

The library only has headers so the shared library overhead is not worth
it.

Test: m
Bug: 280829178
Change-Id: Ie138d1be95aaad9b230407cf03a46d182dd2a307

fout is double checked for nullptr instead of ferr which forces
to call fclose on null ferr.

Test: atest incidentd_test
Test: m incidentd_service_fuzzer && adb sync data && adb shell /data/fuzz/x86_64/incidentd_service_fuzzer/incidentd_service_fuzzer
with clusterfuzz testcase
Bug: 283453191

Change-Id: I3a4912cf70f4f32cb66fb870febbbfff1a7c6adc

idmap2d is being often called for the same targets, e.g. system
apps. Caching those apks makes idmap verification and creation
much faster e.g. for user switching

Test: build + boot + UTs
Bug: 271904589
Change-Id: Ib6f7af385c2389b50d5c74aa08b4bab290580809

When random parcel is sent to incidentd, AIDL service
crashes with null pointer dereference.

Test: atest incidentd_test
Test: m incidentd_service_fuzzer && adb sync data && adb shell /data/fuzz/x86_64/incidentd_service_fuzzer/incidentd_service_fuzzer
Bug: 282963041
Change-Id: I0e0ebc479acbfd3f5e9f56b629380b5d90021f45