Add missing error checks in tests.

To ensure we don't get tests spuriously passing when setup has failed.

Test: atest installd_service_test
Bug: 78442602
Change-Id: I32c8dc4f3d2b9d6dc0f289b170fd13198c87cac3

Merge "installd_service_test: fix ServiceTest.CreateAppDataSnapshot_ClearsCache"

installd_service_test: fix ServiceTest.CreateAppDataSnapshot_ClearsCache

This test is failing on cf_x86_phone-userdebug target, with an error:
Failed copying /data/local/tmp/user_de/0/com.foo to /data/local/tmp/misc_de/0/rollback'

Logs shows that cp fails with "No such file or directory error":
02-05 08:37:49.444 12746 12746 I cp      : cp: /data/local/tmp/misc_de/0/rollback: No such file or directory
02-05 08:37:49.445 12746 12746 I cp      : cp terminated by exit(1)

Testcase is failing, because it doesn't create
/data/local/tmp/misc_de/0/rollback directory. It is hard to spot, because
other testcases in installd_service_test also create that directory, meaning
that depending on the order in which testcases are run,
CreateAppDataSnapshot_ClearsCache will either fail or succeed.

Verified that this is the case by running only
CreateAppDataSnapshot_ClearsCache testcase:
./data/nativetest/installd_service_test/installd_service_test --gtest_filter=*CreateAppDataSnapshot_ClearsCache*

Bug: 123631551
Fixes: 123631551
Test: ./data/nativetest/installd_service_test/installd_service_test --gtest_filter=*CreateAppDataSnapshot_ClearsCache*
Change-Id: I8277efe6076bac28fd07ea3705a21aae6b8cab14

Merge "Use sh_binary instead of cc_prebuilt_binary for shell scripts"

Use sh_binary instead of cc_prebuilt_binary for shell scripts

Bug: 123829926
Test: treehugger
Change-Id: If9d2da84b30aebf8881c6da12efbe99549f5e0cc

Merge "Have `otapreopt_chroot` bind-mount Bionic artifacts from the Runtime APEX."

Merge "Add handling for PH property dex2oat_resolve_startup_strings"

Merge "Add "ls -l /data/anr" output to bugreport"

Add handling for PH property dex2oat_resolve_startup_strings

Added unit test to verify the property is passed to the oat file.

Test: adb shell /data/nativetest64/installd_dexopt_test/installd_dexopt_test
Bug: 116059983
Bug: 123524494

Change-Id: Ib4fe5401d58c6ac05ef3011bef04e38d1b6d4ba9

Add "ls -l /data/anr" output to bugreport

Bug: 123720446
Test: with adb bugreport
Change-Id: I2a00d370b8e61b4f0059cdf2d8d67136968a9f20

Merge "Refactor the APEX package activation/deactivation logic in `otapreopt_chroot`."

Have `otapreopt_chroot` bind-mount Bionic artifacts from the Runtime APEX.

Have `otapreopt_chroot` bind-mount Bionic artifacts (`linker(64)`,
`libc.so`, `libm.so`, `libdl.so`) from the Runtime APEX
(`/postinstall/apex/com.android.runtime`) into `/postinstall/bionic/`,
much the same way `init` sets up Bionic bind-mounts in the default
mount namespace.

Test: m otapreopt_chroot
Test: A/B OTA update test (asit/dexoptota/self_full).
Bug: 113373927
Bug: 120266448
Change-Id: Ie254a94ae35e62f88375bc89380456f15796566a

Refactor the APEX package activation/deactivation logic in `otapreopt_chroot`.

This is in prevision of reusing the package deactivation logic as a
clean-up step upon Bionic bind-mounting failure.

Test: m otapreopt_chroot
Test: A/B OTA update test (asit/dexoptota/self_full).
Bug: 113373927
Bug: 120266448
Change-Id: I17b2d57252af7aca1fc27b4c051d6cc9d373c223

Merge "installd: remove restored ce_data if de_data rollback failed"

Merge "installd: Add destroyAppDataSnapshot and corresponding binder API"

installd: Add destroyAppDataSnapshot and corresponding binder API

- destroyAppDataSnapshot: deletes both ce and de snapshots (controlled
  via storageFlags), for ce snapshot fallbacks to deleting it by it's
  inode in case snapshot is not accessible (user hasn't unlocked the
  phone yet).
- snapshotAppData: in case ce snapshot was requested, it will now return
  inode if a folder with the snapshot. Otherwise it will return 0.

Passing ceSnapshotInode to destroyAppDataSnapshot should make logic
inside RollbackManagerService a bit easier. Since it will be able to
delete ce snapshot regardless of user unlocking their phone.

Bug: 112431924
Test: installd_service_test installd_utils_test
Change-Id: I015de12593b87f3000f004cc0a901a9208d0b693
Merged-In: I015de12593b87f3000f004cc0a901a9208d0b693

Merge "Installd: Refactor binder tests"

Merge "libbinder_ndk: ScopedAResource into impl namespace"

Installd: Refactor binder tests

Add helper macros to automatically print return status errors.

Bug: 123631551
Test: atest --test-mapping frameworks/native/cmds/installd
Merged-In: I4ab448fabbab91cc608c318a07f9bf95547062fc
Change-Id: I4ab448fabbab91cc608c318a07f9bf95547062fc

Merge "Fix regression in startBugreport flow"

Merge "Ensure only one bugreport is in progress in dumpstate"

Merge "installd: Create "lib" symlink with the correct label."

Merge "Add OWNERS to inputflinger"

Merge "libbinder_ndk: remove draft status"

Add OWNERS to inputflinger

Test: N/A
Change-Id: If16fbe675557cb01770bdc2d6cbc55c46c384d9c

installd: Create "lib" symlink with the correct label.

When installing an application which has a 32 bit ABI, system_server
(via installd) creates a compatibility "lib" symlink in the application
home directory. See:

https://android.googlesource.com/platform/frameworks/base/+/d5d7492040c1730899cccef9916541176004635c/services/core/java/com/android/server/pm/PackageManagerService.java#22876
and
https://android.googlesource.com/platform/frameworks/native/+/6b8e52c805f124f8b1d7e511ae68d01d0769c32b/cmds/installd/InstalldNativeService.cpp#2077

When a process creates a filesystem object within a directory, in
inherits the directory type, but DOES NOT inherit the directory MLS
categories. See

* https://www.spinics.net/lists/selinux/msg21893.html
* https://www.spinics.net/lists/selinux/msg21897.html

for more details on this behavior.

Without subsequent fixup, an installd created symlink in an
application home directory will have incorrect SELinux MLS categories,
and as a result, may be unreadable to the application.

Modify installd to assign the "lib" label the same MLS categories as
the enclosing parent directory.

Steps to reproduce:

  1) adb shell
  2) su
  3) ls -laZ /data/data/*/lib

Expected:

  crosshatch:/ # ls -laZ /data/data/*/lib
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0:c177,c256,c512,c768 46 2019-01-30 12:46 /data/data/com.android.chrome/lib -> /system/product_services/app/Chrome/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0:c512,c768           38 2019-01-30 12:46 /data/data/com.android.omadm.service/lib -> /system/priv-app/DMService/lib/arm
  lrwxrwxrwx 1 root root u:object_r:privapp_data_file:s0:c512,c768       54 2019-01-30 12:46 /data/data/com.android.vending/lib -> /system/product_services/priv-app/Phonesky/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0:c141,c256,c512,c768 30 2019-01-30 12:46 /data/data/com.google.android.apps.tycho/lib -> /product/app/Tycho/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0:c512,c768           46 2019-01-30 12:46 /data/data/com.google.android.videos/lib -> /system/product_services/app/Videos/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0:c180,c256,c512,c768 58 2019-01-30 12:46 /data/data/com.google.android.webview/lib -> /system/product_services/app/TrichromeWebView/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0:c512,c768           34 2019-01-30 12:46 /data/data/com.qti.ltebc/lib -> /system/app/QAS_DVC_MSP/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0:c512,c768           38 2019-01-30 12:46 /data/data/com.qualcomm.ltebc_vzw/lib -> /system/app/QAS_DVC_MSP_VZW/lib/arm

Actual:

  crosshatch:/ # ls -laZ /data/data/*/lib
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0     46 2019-01-30 12:36 /data/data/com.android.chrome/lib -> /system/product_services/app/Chrome/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0     38 2019-01-30 12:36 /data/data/com.android.omadm.service/lib -> /system/priv-app/DMService/lib/arm
  lrwxrwxrwx 1 root root u:object_r:privapp_data_file:s0 54 2019-01-30 12:36 /data/data/com.android.vending/lib -> /system/product_services/priv-app/Phonesky/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0     30 2019-01-30 12:36 /data/data/com.google.android.apps.tycho/lib -> /product/app/Tycho/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0     46 2019-01-30 12:36 /data/data/com.google.android.videos/lib -> /system/product_services/app/Videos/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0     58 2019-01-30 12:36 /data/data/com.google.android.webview/lib -> /system/product_services/app/TrichromeWebView/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0     34 2019-01-30 12:36 /data/data/com.qti.ltebc/lib -> /system/app/QAS_DVC_MSP/lib/arm
  lrwxrwxrwx 1 root root u:object_r:app_data_file:s0     38 2019-01-30 12:36 /data/data/com.qualcomm.ltebc_vzw/lib -> /system/app/QAS_DVC_MSP_VZW/lib/arm

Bug: 123350324
Test: manual
Change-Id: Id09846556cb0ba7e39fbc57f9039f072d6a752a1

libbinder_ndk: ScopedAResource into impl namespace

This is an implementation detail that may change in the future.

Fixes: 123601615
Test: atest android.binder.cts
Change-Id: I006d8bf8b64228a2cc27bee671b6a9838e8df492

Fix regression in startBugreport flow

Ensure what's passed to the background thread is around.

BUG: 123617758
Test: adb shell /data/nativetest64/dumpstate_test/dumpstate_test
Test: manual test of API via a test client
Change-Id: I161c0d378c04cf925778fdb8712b1bd68901fb91

Merge "Use screenshot file descriptor if available"

Merge "Fix/suppress google-explicit-constructor warnings"

Fix/suppress google-explicit-constructor warnings

* Use NOLINT or NOLINTNEXTLINE to suppress warnings on intended converters

Bug: 28341362
Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,google-explicit-constructor
Change-Id: I5c73437ecee0d672863664cc8c2970b099597f77

Merge "installd: clear app's cache before doing a snapshot"

libbinder_ndk: remove draft status

So that it is included in future NDK releases.

Fixes: 123299014
Test: N/A
Change-Id: I118b3190ce01147af84a33156ac01cdcde54a20d

Merge "Restore security context of `/postinstall/apex` earlier in otapreopt_chroot."

Use screenshot file descriptor if available

Test: adb shell /data/nativetest64/dumpstate_test/dumpstate_test
BUG: 123576130
Fixes: 123576130

Change-Id: I2ac2dc3bfc7920a00e4148ca29169c171558de3a

Ensure only one bugreport is in progress in dumpstate

BUG: 123571915
Test: current bugreport flows work
Test: interactive bugreport
Test: adb shell /data/nativetest64/dumpstate_test/dumpstate_test
Test: adb shell /data/nativetest64/dumpstate_smoke_test/dumpstate_smoke_test
Change-Id: I1145a27ed6dde43142789d876902e32d1a286bb2

Merge "Convert Android.mk file to Android.bp"

installd: clear app's cache before doing a snapshot

Bug: 112431924
Test: installd_service_test
Change-Id: I4a6c3bbf071dcee007e48ece40e5009c1a43ecca

Merge "installd: Validate volumeUuid in snapshotAppData and restoreAppDataSnapshot"

Merge "Add vendor dumpsys."

Restore security context of `/postinstall/apex` earlier in otapreopt_chroot.

Invoke `selinux_android_restorecon` on `/postinstall/apex` just after
mounting a tmpfs filesystem in it, so that this directory is correctly
labeled (with type `postinstall_apex_mnt_dir`) and may be manipulated
in following operations (`chmod`, `chown`, etc.) following updated
policies restricted to `postinstall_apex_mnt_dir` (instead of
`tmpfs`).

Test: m otapreopt_chroot
Test: A/B OTA update test (asit/dexoptota/self_full).
Bug: 113373927
Bug: 120796514
Change-Id: I0b243a00e0443e439afda055d3b12aa9eefe0503

Merge "Update filesystem permissions for task tracepoints."

installd: Validate volumeUuid in snapshotAppData and restoreAppDataSnapshot

Only supported values are nullptr and "TEST".

Bug: 112431924
Test: installd_service_test
Change-Id: I27260a3554d009053206d8e3d1e213306ff3d7d5

Update filesystem permissions for task tracepoints.

These tracepoints are used for memory debugging, and should be enabled
for user builds.

Bug: 123367055
Test: manual: took a trace with Traceur and verified that these
tracepoints appear.

Change-Id: I31fafc11d60a96f2a4c58f2670decb2480c0ec78
Merged-In: I31fafc11d60a96f2a4c58f2670decb2480c0ec78

Merge "Remove unused DumpstateOptions parcel"

Merge "installd: clear existing snapshots before performing a new one"

Remove unused DumpstateOptions parcel

Now using bugreport mode to indicate runtime options.

BUG: 111441001
Test: adb shell /data/nativetest64/dumpstate_test/dumpstate_test
Change-Id: I1890323394a5f146a8e0e971914b8b7437e75547

Merge "Load libandroid_runtime lazily."

Add vendor dumpsys.

$ adb shell /vendor/bin/dumpsys -l
<hangs>
$ adb shell /vendor/bin/sh -c "/vendor/bin/dumpsys -l"
Currently running services:
...
$ adb root
$ adb shell /vendor/bin/dumpsys -l
Currently running services:
...

Bug: 123433249
Test: manual (see above)
Change-Id: I4d74be3ef43e7d65154be27c4cf85ab32e09dfef

Load libandroid_runtime lazily.

Memory/performance for low-level processes that do not need this.

Bug: 118783779 # follow-up to cleanup
Bug: 123284997
Test: atest android.binder.cts
Change-Id: Ie8de0a082dfb75786a45a448185fedb317daa94e

Convert Android.mk file to Android.bp

See build/soong/README.md for more information.

Bug: 122332691
Test: treehugger
Change-Id: I2146596aad99baa128511d07ab73470885319d49

Merge "Add jwcai and tianyuj to the owner's file of libs/ui"

Merge "DO NOT MERGE: Revert "Revert "Add libprocessgroup dependency and fix sched_policy include"""

DO NOT MERGE: Revert "Revert "Add libprocessgroup dependency and fix sched_policy include""

This reverts commit f145309d9c1549b06029bb00a3d5fe2bf459ed02.

Reason for revert: AOSP is fixed with new vendor image

Change-Id: If522adf1ea36e7792b1ae26234ec2bdb2756bdfd
Signed-off-by: Suren Baghdasaryan <surenb@google.com>

installd: remove restored ce_data if de_data rollback failed

I think it's clearer (and safer) if failed rollback leaves app devoid of
any data rather that with only ce_data.
LMK if you disagree, I will just abandon this change then. :)

I tried adding a test to make de_rollback fail (while keeping
ce_rollback successful), but failed to.
Only idea I have in mind is to intercept calls to /system/bin/cp, which
doesn't look like a unit test anymore (and also I'm not sure how
feasible it is).

Test: existing tests in installd_service_test pass
Bug: 112431924

Change-Id: I3c6214d70bac682874ea846a01294b5361470f93

installd: clear existing snapshots before performing a new one

Bug: 112431924
Test: installd_service_test
Change-Id: I23a97f113e8a7423a7c509ee06d1436a2c2c961f

Merge "Refactor paths to Android Runtime binaries in directory `cmds/installd/`."

Merge changes from topics "runtime-apex-otapreopt_chroot-tear-down", "runtime-apex-installd-support", "runtime-apex-otapreopt_chroot-support"

* changes:
  Implement a tear-down phase in otapreopt_chroot.
  Revert^2 "Have installd use ART binaries from the Android Runtime APEX."
  Revert^2 "Have `otapreopt_chroot` mount APEX packages in the chroot dir."

Merge "Use the consent service to check if user has consented to taking a bugreport"

Use the consent service to check if user has consented to taking a
bugreport

Test: verified current flows of bugreport generation still work
Test: TBD test the new flow
Change-Id: I5fc154905515db664f76d1453c9d79364dc2f01e

Merge "Add IIncidentCompanion binder API."

Merge "installd: Implement support for userdata snapshot and restore."

Merge "DO NOT MERGE: Revert "Add libprocessgroup dependency and fix sched_policy include""

DO NOT MERGE: Revert "Add libprocessgroup dependency and fix sched_policy include"

This reverts commit dc9078de949983a91b04dedc78e4089645f8626d.

Reason for revert: Broke AOSP

Change-Id: Ia5cb6e8f829669a6e0dd01260605b986a1282c85
Signed-off-by: Suren Baghdasaryan <surenb@google.com>

Merge "Add FEATURE_TELEPHONY_IMS feature definition"

Add FEATURE_TELEPHONY_IMS feature definition

Add an IMS feature definition so that telephony doesn't
bring up unnecessary resources or run GTS tests on a device
that doesn't support it.

Bug: 118823723
Test: manual

Change-Id: I359f9e337ec785745f5d0ca778615d12c61faedf

Add IIncidentCompanion binder API.

It's in frameworks/native because dumpstate uses it.

BUG: 111441001
Test: builds
Change-Id: I0897bec8882395d3444f46a145e9f91f4e9da32d

Refactor paths to Android Runtime binaries in directory `cmds/installd/`.

Have `dexopt` and `otapreopt` use the same set of definitions.

Test: A/B OTA update test (asit/dexoptota/self_full).
Bug: 113373927
Bug: 120796514
Change-Id: I94e55a831c68b67cb81cfa91281b2feb0cc4b365

Merge "[view compilation] Add viewcompiler support to installd"

Merge "Dumpstate aidl: add cancelBugreport"

Dumpstate aidl: add cancelBugreport

This allows adding a cancel method to the bugreporting API. Note that it
is implemented on the java side.

Also removed unused args from onFinished.

BUG: 111441001
Test: adb shell /data/nativetest64/dumpstate_test/dumpstate_test

Change-Id: I6a3a0d652c89dc157de4a4056ccddc40e5d6e549

Add jwcai and tianyuj to the owner's file of libs/ui

for the development of BufferHub project. OWNERS file is out of sync.
This change syncs the file and added jwcai and tianyuj to the OWNERS.

Bug: None
Test: None
Change-Id: I817d3dc323e46aa7417a583d1faa8e0bbb87c3df

Merge "Add a flag that makes dumpstate wait for options"

Add a flag that makes dumpstate wait for options

There are 2 ways of running bugreports. Running dumpstate binary
directly with command line options (existing flow), or running the
binary to start the binder service, which waits for a caller to supply
arguments like file descriptors to write output to (new flow to support
the bugreport triggering API).

Support the second flow with a new commandline flag.

Also add a new service that uses the flag, to be used in the API.

BUG:111441001
Test: adb shell setprop "ctl.start" "bugreportd" -> starts service &
waits.
Test: interactive bugreport still works
Change-Id: I64cd03d15f45a896692d7462998d4faa6e413ef1

Merge "Add libprocessgroup dependency and fix sched_policy include"

installd: Implement support for userdata snapshot and restore.

Adds two binder APIs:
- snapshotAppData: Makes a copy of the apps credential encrypted
  and/or device encrypted data for a given user to one of
  /data/misc_[ce|de]/rollback. This path is writable only by
  installd.
- restoreAppDataSnapshot: Restores a copy of the apps credential
  encrypted and/or device encrypted data from a snapshot, if one
  exists.

Implemented for use from RollbackManagerService, which will make
a transient copy of an app package as well as its data in order to
roll them back within a specified window.

Bug: 112431924
Test: installd_utils_test, installd_service_test

Change-Id: I3e4d36c11e52fb885b585b1946e215cf986206fd
Merged-In: I3e4d36c11e52fb885b585b1946e215cf986206fd

Implement a tear-down phase in otapreopt_chroot.

Collect the list of APEX packages activated by `otapreopt_chroot` and
deactivate them after executing `otapreopt` (in its own process).

Also share android::installd::Exec between `otapreopt` and
`otapreopt_chroot`.

This tear-down phase ensures loop devices associated during by
`otapreopt_chroot` are detached before `otapreopt_chroot` exits.

Test: m otapreopt_chroot
Test: A/B OTA update test (asit/dexoptota/self_full).
Bug: 113373927
Bug: 120796514
Change-Id: I236ac49bb698fb672e7a4f9de9903229e535e9c6

Revert^2 "Have installd use ART binaries from the Android Runtime APEX."

This reverts commit 3a77febd0b57dcaceee2bb13b4bccddffb25ffc8.

Test: Device boot test (with Android Runtime APEX).
Test: A/B OTA update test (asit/dexoptota/self_full).
Bug: 113373927
Bug: 120796514
Change-Id: Ib22f91b5f8916f13b31bcd630600a86b9d86e25d

Revert^2 "Have `otapreopt_chroot` mount APEX packages in the chroot dir."

We need at least the Android Runtime APEX to be able to run `dex2oat`
in `otapreopt`. For simplicity, reuse `apexd`'s logic and mount all
the APEX packages found on the system partition in /postinstall/apex.

This is a temporary change. The final design will integrate A/B OTA
updates and `apexd`.

This reverts commit 00b5b3f01b7b6bf96d0d66d650ede8aa4734bf3f.

Test: m otapreopt_chroot
Test: A/B OTA update test (asit/dexoptota/self_full).
Bug: 113373927
Bug: 120796514
Change-Id: Ifa45c282d04ae443e55bd41d8c9cbb4ea5f837d4

Add libprocessgroup dependency and fix sched_policy include

surfaceflinger, installd and libvrflinger are using set_sched_policy
which is moved into libprocessgroup. Add libprocessgroup dependency
and include required headers.

Exempt-From-Owner-Approval: janitorial

Bug: 111307099
Test: builds, boots

Merged-In: Iccb7c54a85ca148d44ce687fe05e89ad4369e104
Change-Id: Iccb7c54a85ca148d44ce687fe05e89ad4369e104
Signed-off-by: Suren Baghdasaryan <surenb@google.com>

Merge "Remove outfile option from dumpstate"

Merge "Fix dumpstate test instructions"

Merge "Delay start of surfaceflinger until after the Runtime APEX is mounted."

Delay start of surfaceflinger until after the Runtime APEX is mounted.

It depends on libdexfile_external which is provided by the Runtime APEX.

Test: flash and boot
Test: atest CtsJdwpTestCases
Bug: 113373927
Change-Id: Ic3af21d74854cce02180d3d2e9e3b1b338456e46
Exempt-From-Owner-Approval: Trying this CL out for gathering numbers - will revert if substantial.

Remove outfile option from dumpstate

Remove the ability to write to any directory. Instead support only
writing to its internal directory (currently /bugreports, which points
to Shell app's directory), or a caller-specified file fd.

We cannot expect all callers to supply an fd, because in the poweruser
case where the bugreport is triggered with combo keys, the API is
bypassed and dumpstate binary will be run.

This should be a safe change since sepolicy should not allow dumpstate
to write to arbitrary directories anyway.

This keeps the API lean and keeps the user consent for sharing more
focused.

Note that the current callers all pass in /data/user_de/0/com.android.shell/files/bugreports/bugreport
as the outfile argument already, which is the location /bugreports
symlink points to, so it should work just as before.

BUG:111441001
Test: adb bugreport
Test: adb shell bugreport
Test: interactive bugreport
Test: adb shell /data/nativetest64/dumpstate_test/dumpstate_test
Change-Id: Iae8593dc4745147b7bdae25738fcd69b3c20aaf0

Fix dumpstate test instructions

Amend command to copy all tests including smoke tests
Test: verified instruction

Change-Id: I107be21f2acab3a9965a114fb2219e3d21c871b1

Merge "Add calling uid and calling package to startBugreport"

Add calling uid and calling package to startBugreport

This is so they can be passed to the consent service which will present
the user with a dialog and get their consent to share the bugreport with
the calling package.

Also update a few TODOs.

BUG: 111441001
Test: builds
Change-Id: I0a72c8e2d697f8a25f307cfce73c8f03ed1746e2

[view compilation] Add viewcompiler support to installd

This change adds methods to installd to call viewcompiler and generate
precompiled layouts when requested either by `pm compile --compile-layouts`,
package install, or package upgrade.

Bug: 111895153
Test: manual
Change-Id: Ic021d7a41c15642664f44542653170ad24055f22

Merge "Fix typo cpu6 -> cpu7"

Fix typo cpu6 -> cpu7

This prevented perfetto from clearing the ftrace buffer for cpu7
(and only cpu7). Seems a copy paste error.

Test: Untested.
Bug: 122609457
Change-Id: I2096dc333880d94cb25ba5ce50272444e8ae999e

Merge "Remove ftrace dumping from dumpstate"

Merge "Fix/suppress sensor google-explicit-constructor warnings"

Remove ftrace dumping from dumpstate

When capturing a bug report, dumpstate checks if ftrace
is on, and if so, steals the trace from the buffer.
This feature doesn't seem to be actually used, getting
rid of unnecessary complexity and unnecessary permissions.
Furthermore, stealing the trace is bad for iorapd, studio
perfetto, Traceur and all the other trace users.
Matching selinux change: aosp/875395

Bug: 122987614
Test: run dumpstate before and after patch,
      file sizes are comparable,
      observed no tracing-related errors.
Change-Id: I8bcd478c9bde776a2ec21b5e9a97aedaaa3681fd

Fix/suppress sensor google-explicit-constructor warnings

* Add explicit to conversion constructors/operators
* Use NOLINT or NOLINTNEXTLINE to suppress warnings on intended converters

Bug: 28341362
Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,google-explicit-constructor
Change-Id: I63e196797c287182fe8ccdad0e896f864a10bf76

Merge "Add filemap events for iorapd"

Merge "NFC: Switch ATRACE_BUFFER_INDEX to be a more friendly macro statement."

Merge "Call new dumpstate listener methods"

Merge "Move to C API of libnativeloader."

Call new dumpstate listener methods

The old listener methods are still connected, but will be removed soon.

Also fixed the progress debug logs, which rarely work since the guard
condition is rarely met.

BUG: 111441001
Test: adb shell /data/nativetest64/dumpstate_test/dumpstate_test
Change-Id: I29b95685df89d0bbebde7d95d496500bf51f4aaa