2ff42d264bc91b2785af404975753767406b0620 - android_packages_modules_Connectivity

commit	2ff42d264bc91b2785af404975753767406b0620	[log] [tgz]
author	Junyu Lai <junyulai@google.com>	Thu Dec 07 16:57:13 2023 +0800
committer	Junyu Lai <junyulai@google.com>	Wed Jan 10 13:56:55 2024 +0800
tree	dc6aab76a54cfe2696f817eccd38e8d90183b1f6
parent	5b96f865cd07c7cd065a252402174249c2bd636a [diff]

Migrate Tethering#dump to use runWithScissorsForDump

This change includes:
  1. Refactor the logic in Tethering#dump into utils class.
  2. Move the utils class to a common place which could be referenced
     from other sub-modules.
  3. Add @MonitorThreadLeak annotation to enforce
     there is no thread leak problem.

Test: atest FrameworksNetTests NetworkStaticLibTests
Test: atest ConnectivityCoverageTests:com.android.networkstack.tethering.TetheringTest#testDumpTetheringLog
Test: adb shell dumpsys tethering (with hardcoded exception)
Fix: 312669345

Change-Id: Ia6fdfeeec2110afa0ec9e056e9db3843748845c3

Tethering/src/com/android/networkstack/tethering/Tethering.java[diff]
Tethering/tests/unit/src/com/android/networkstack/tethering/TetheringTest.java[diff]
service/src/com/android/server/ConnectivityService.java[diff]
service/src/com/android/server/connectivity/HandlerUtils.java[Deleted - diff]
staticlibs/Android.bp[diff]
staticlibs/device/com/android/net/module/util/HandlerUtils.java[Added - diff]
staticlibs/tests/unit/src/com/android/net/module/util/HandlerUtilsTest.kt[Renamed from tests/unit/java/com/android/server/HandlerUtilsTest.kt - diff]

7 files changed

tree: dc6aab76a54cfe2696f817eccd38e8d90183b1f6