71b51538982673feca0913ba9325a47869f9694c - android_packages_modules_Connectivity

commit	71b51538982673feca0913ba9325a47869f9694c	[log] [tgz]
author	Junyu Lai <junyulai@google.com>	Thu Feb 01 10:39:01 2024 +0800
committer	Junyu Lai <junyulai@google.com>	Thu Feb 01 15:24:35 2024 +0800
tree	6dce7ea5879298c1ab6b370df0c1e83f1d67999c
parent	d7773093eeec02804aa0992a8ecbd2cca12f227c [diff]

Rename check*Permission* to has*Permission

The check*Permission methods are a bit error-prone because people
can forget to check the return value and expect that they behave
like the enforce* methods. This was pointed out before in some bugs.

This change adds @CheckResult annotation to enforce linter warning
and rename methods accordingly.

Test: TH
Fix: 279691948
Change-Id: Ibb0df6540fe0ed40289bb190b790130daaa2f763

framework-t/src/android/net/NetworkStatsAccess.java[diff]
service-t/src/com/android/server/NsdService.java[diff]
service-t/src/com/android/server/ethernet/EthernetServiceImpl.java[diff]
service-t/src/com/android/server/net/NetworkStatsService.java[diff]
service/src/com/android/server/ConnectivityService.java[diff]
staticlibs/framework/com/android/net/module/util/PermissionUtils.java[diff]
staticlibs/tests/unit/src/com/android/net/module/util/PermissionUtilsTest.kt[diff]
tests/unit/java/com/android/server/ConnectivityServiceTest.java[diff]

8 files changed

tree: 6dce7ea5879298c1ab6b370df0c1e83f1d67999c