Age | Commit message (Collapse) | Author |
|
Move IAppOpsCallback to frameworks/native for use in both java and
native.
Fix-up callsites with fully qualified package name and the correct
arguments.
Test: compiles
Test: atest CtsMediaAudioPermissionTestCases
Test: atest service.incremental_test
Bug: 322692565
Flag: EXEMPT mechanical
Change-Id: I8d5fff5ef86e0b2d68df4a95745fe430134fe1f0
|
|
frameworks/base/services/incremental/IncrementalService.cpp:2004:29: warning: ISO C++ requires field designators to be specified in declaration order; field 'signature' will be initialized after field 'metadata' [-Wreorder-init-list]
2004 | .metadata = {targetLibPath.c_str(), (IncFsSize)targetLibPath.size()},
| ~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
frameworks/base/services/incremental/IncrementalService.cpp:2002:30: note: previous initialization for field 'signature' is here
2002 | .signature = {},
| ^~
Test: presubmit
Change-Id: I7c71ed46f36f30f3bff947e513b6a021d1c61b8c
|
|
As part of go/test-module-config we are moving test options from
TEST_MAPPING -> Android.bp files.
In previous Cls, we created the new `test_module_config` rules in
Android.bp
This is updating the TEST_MAPPING file to use those rules.
It is also removing "FlakyTest and IgnoreTest" exclude annoations as
they are now added in gcl files per run rather than ad-hoc per module.
I have a script that looks at the generated tradefed config file for the
new options added in Android.bp files, then it looks at TEST_MAPPING
files and find the places to update where the options match for the
test.
I am also doing abtd runs of each TEST_MAPPING file before and after my
change and ensuring the number of tests run is the same (or at least as
many). There are cases where tradefed would comping include-filters
across TEST_MAPPING entries for the same module, but now they will be
purposefully split up, causing some tests to be run under two different
modules.
Flag: TEST_ONLY
Test: Ran adbt on each TEST_MAPPING and compared before and after
results. Verified we were still running all the tests we were before.
i.e. after the adbt run, I would download the test artfiact for the
tradefed detailed evenvt and compare test counts.
You can see CtsAppTestCases became CtsAppTestCases_cts_requesttileserviceaddtest, etc.
I'm not including results for all 100 TEST_MAPPING files, but I did
verify with scripts and eyes.
Minor differences (like 2011 vs 2009) on a test that didn't change are
ignored, but in general there were more tests run, not fewer.
% diff <(grep started frameworks_base_services_core_java_com_android_server_statusbar_TEST_MAPPING/BASE.details) <(grep started frameworks_base_services_core_java_com_android_server_statusbar_TEST_MAPPING/NEW.details) | grep run
< [run x86_64 CtsAppTestCases (testCount: 6,attempt: 0) started]
> [run x86_64 CtsAppTestCases_cts_requesttileserviceaddtest (testCount: 6,attempt: 0) started]
< [run x86_64 CtsLocationFineTestCases (testCount: 96,attempt: 0) started]
> [run x86_64 CtsLocationFineTestCases_android_server_location (testCount: 96,attempt: 0) started]
< [run x86_64 FrameworksNetTests (testCount: 2009,attempt: 0) started]
> [run x86_64 FrameworksNetTests (testCount: 2011,attempt: 0) started]
Test-Mapping-Slo-Bypass-Bug: b/335015078
Change-Id: If6e3cd0624ac1c16f1cd088566d967769f47199c
|
|
Change-Id: Icdf645793dbe891d76adf78d70b9cb5287e510e4
|
|
* changes:
Fix fs-verity API for secondary users
Fix build for ag/25776653
Update @UsesReflection annotation
Merge Android 24Q1 Release (ab/11220357)
|
|
Change-Id: I87e21b50a247f82d65a3cd46cd1612676d9dcbe0
|
|
83933ce167 am: a15a3a80a2
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2859205
Change-Id: I85a3fb0d65ba566c5eddb600133f45395089f6fb
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Test: presubmit
Change-Id: Ifd5f633a517d6dbd7344849a739acb32cb142296
|
|
Platform_testing LargeTest annotation is for
host side tests, device
tests should use androidx annotation.
Bug: 293837621
Test: build
Change-Id: I8ea6f256b542a4ed0030b3d366c37f6fdafa6b3b
|
|
main
|
|
BUG: 293380233
Test: presubmit
Change-Id: I3519bf2260a4c967a033cb4330e41a86f1a124e4
|
|
To unblock presubmit.
Bug: 292174535
Test: atest CtsPackageManagerTestCases
Change-Id: I729156f85c072b62c25043b6eabf44616e689239
|
|
libcxx implements wait_until using wait_for, which in
its turn has to call a syscall that's basically wait_until,
causing a double conversion of the waiting time parameter.
Because of this, infinity detection code in those conversions
sometimes fail if the delat between the conversions is long
enough - causing a syscall error for 'invalid time'
This CL explicitly calls non-timed wait() instead.
Bug: 293223125
Bug: 292138960
Test: build + unit tests + presubmits
Change-Id: Id30fdfbe374b63fcaed627fcdf374f1e501a7807
|
|
Bug: 288461387
Bug: 288323752
Test: presubmit
Change-Id: I453a4151100ca42495e0f2473e5599dec38ab70d
|
|
Bug: 288460044
Test: atest --test-mapping frameworks/base/services/incremental/TEST_MAPPING
Change-Id: Idc5cff6e470ad652835624f9bab66398f8bfb0f5
|
|
already included that way in another location." am: 12298c9002 am: 9d70d31cd1 am: c2e6d8df4e
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2460715
Change-Id: I1dd7ebb5f654e55ecf7c86f4ceb5713fdc698b61
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
already included that way in another location." am: 12298c9002 am: 9d70d31cd1
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2460715
Change-Id: Ide0d750eb2d00c3a9c8f270af9a4b727e3645034
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
included that way in another location.
No need to carry that test exactly the same in both presubmit & presubmit-large
Change-Id: I843339f1583e1a163894c52978bdcb4cc9f8ffa6
Test: presubmit
Bug: 270214959
|
|
|
|
This should prevent apps from circumventing the GET_APP_METADATA
permission by reading the file directly if they are aware of the file
path.
Bug: 267823160
Test: atest android.packageinstaller.install.cts.InstallAppMetadataTest
Change-Id: I4aab10b48e62234bc252535ab2e2c8b9c77a7ac3
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/21342999
Change-Id: I988c7ac970d185bbadd49889d786e990334c111e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
|
|
Bug: 263449831
Fixes: 263449831
Test: atest service.incremental_test
Change-Id: Ia3da6ca047c46807cbd72ef797bb4b300c38efb3
|
|
Bug: 263449831
Fixes: 263449831
Test: atest service.incremental_test
Change-Id: Ia3da6ca047c46807cbd72ef797bb4b300c38efb3
|
|
Bug: 263449831
Test: adb shell dumpsys incremental
Change-Id: Ie6960d22c1d0e018b50de1c9e431c403fe442458
|
|
Bug: 258819618
Test: atest :kernel-presubmit
Change-Id: Ie887d7f11c2da982e224eb55ef61dfb95a23b438
Signed-off-by: Edward Liaw <edliaw@google.com>
|
|
|
|
In newer releases of the protobuf library, the release_* methods
are marked [[nodiscard]], which causes a compilation error here.
Bug: 203713560
Test: presubmit
Change-Id: If8c6392a5d36014f1839deaaf241b636610add71
|
|
In newer releases of the protobuf library, the release_* methods
are marked [[nodiscard]], which causes a compilation error here.
Bug: 203713560
Test: presubmit
Change-Id: I44a9d52730bdc199e34cb25be959a955fff8f128
|
|
The out-of-date std::span in external/libcxx currently uses ptrdiff_t
as the type of size() and the second ctor argument. The C++20 std::span
actually uses size_t for these values instead. Try to accommodate both
versions of std::span until libc++ is upgraded.
Bug: http://b/175635923
Test: treehugger
Change-Id: Idc43f880b6eb21dad79e79fa456847d5865a1ed2
|
|
This will allow tests to disable verification on case-by-case basis.
Bug: 224017187
Bug: 220086205
Test: atest PackageManagerShellCommandTest ResourcesHardeningTest PackageManagerShellCommandIncrementalTest ChecksumsTest
Change-Id: Ie0e77f1cbc52b7fe56e00e8b0b36f47b2422391b
|
|
This patch adds libincfs-test and service.incremental_test in the
presubmit process.
Test: atest service.incremental_test
Test: atest libincfs-test
Test: atest -p system/incremental_delivery/incfs
http://ab/I24700010005521991
Bug: 180650365
Change-Id: Iaea12556d398893875c4ab3bb928477103fa7c18
|
|
44233c693c am: bdf15f2234 am: 65ee23bba4 am: a00b50e072 am: cfa0cf527d
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1810680
Change-Id: I33e5ac11f43881354caf318a68c47cd6f5763ece
|
|
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1810680
Change-Id: I2614171ee4fc1822a2a3c1e7e50658487218a34d
|
|
Bug: http://b/197965342
services.incremental depends on libbinder.so, which already depends on
libutils.so. When linking services.incremental with libutils.a, the
linker tries to open objects from libutils.a to redefine undefined
symbols in libbinder.so. This causes a linker failure with upstream
LLD. Switching to shared libutils.so instead doesn't increase the
dependency closure for services.incremental.
Test: build with aosp/1809741 which has lld with the above behavior.
Change-Id: I2735461ae64ef2e4c0afc170f7b765c1b5b9432e
|
|
The test config takes longer than 900 seconds to run. Move it to a
dedicated group for running slow presubmit Test Mapping test.
Some more context is in the referenced bug, e.g, b/174495337
The group will work exactly the same as presubmit for now.
Bug: 174654670
Bug: 174495337
Test: treehugger
Change-Id: Id9769bbc625b42257af603d69098ae69c0fabb80
|
|
Having a final destructor prevents the class from being inherited from,
adding the designation final to the class makes this behavior more
explicit. This is required to re-enable the error for
-Wno-final-dtor-non-final-class.
Test: m
Merged-In: Ia3126d30e19edfd17f7c8da368e9763ca5501e84
Change-Id: Id1d7c607af9cca0109e1f763052894cf179f4af1
|
|
10secs should be more than enough to stop whatever the DL is doing.
Besides, DL has its own set of IncFS FDs and should not crash.
Bug: 189222575
Fixes: 189222575
Test: atest IncrementalServiceTest
Change-Id: I73cb27d61c7418adeea7536c8263e6ba8c77fd3e
|
|
Plus more robust handling of broken DLs.
Bug: 190012477
Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest com.google.android.packageinstallerv2proxy.host.gts.IncrementalInstallerHostTest
Change-Id: I5cb037d49cd2b140bed1045c99f072112495acfc
|
|
BUG: 187354818
Test: atest service.incremental_test
Change-Id: I632b747440ed4faef31266a3a7b18b7a382ff2cf
|
|
Bug: 186864416
Test: Builds
Change-Id: I04dfc5c2c7143825334ebc62742e335b8c889492
|
|
Bug: 186864416
Test: Builds
Change-Id: I04dfc5c2c7143825334ebc62742e335b8c889492
|
|
+minor fixes
Bug: 187710420
Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest
Change-Id: Iced479532b38bddcd8655df3ce08611434965cf4
|
|
Otherwise we have errors during applyStorageParams:
05-10 15:35:32.363 562 625 E IncrementalService: applyStorageParams failed: Status(-8, EX_SERVICE_SPECIFIC): '-95: '
05-10 15:35:32.363 562 625 E incfs-dataloaderconnector: setStorageParams failed with error: -95
05-10 15:35:32.363 562 625 E incfs-dataloaderconnector: DataLoader supports UID
05-10 15:35:32.367 0 0 E incfs : Can't change sysfs_name mount option on remount
BUG: 187308584
Test: atest CtsContentTestCases:android.content.pm.cts.PackageManagerShellCommandIncrementalTest#testInstallWithIdSigNoMissingPages
Change-Id: Ic2146aa7855e13de9f96794639de556f64e93701
|
|
Bug: 187437944
Test: atest PackageManagerShellCommandTest PackageManagerShellCommandIncrementalTest IncrementalServiceTest PackageManagerServiceTest ChecksumsTest ApkVerityInstallTest
Change-Id: Idc8cb4d170b0bfd98ee3a10bdf57439801739757
|
|
|
|
BUG: 184844615
Test: atest service.incremental_test
Change-Id: Ibbfe68851a4adfd620db9652f1fda550a3c6fb2c
|
|
|
|
into sc-dev
|
|
BUG: 184844615
Test: dumpsys
Change-Id: I3e8178c8a62dc5d09c718b140bb48bda85a3bae5
|