38e149a8e0f922e978f357be35196bfbb635e8ee - LeafOS-Project/android_art

commit	38e149a8e0f922e978f357be35196bfbb635e8ee	[log] [tgz]
author	Jiakai Zhang <jiakaiz@google.com>	Thu Dec 01 15:19:20 2022 +0000
committer	Jiakai Zhang <jiakaiz@google.com>	Thu Dec 08 23:57:42 2022 +0000
tree	a95c69416548561cdaad4f6d517b34afc933b4d1
parent	8745882d469bf045660c7282573e00a9963e5b21 [diff]

Shutdown newFixedThreadPool and newSingleThreadExecutor.

The javadoc of newFixedThreadPool says:
> The threads in the pool will exist until it is explicitly shutdown.

The javadoc of newSingleThreadExecutor says:
> Unlike the otherwise equivalent newFixedThreadPool(1) the returned
> executor is guaranteed not to be reconfigurable to use additional
> threads.

So we have to manually shut them down, or we will leak them.

Bug: 259472777
Test: atest ArtServiceTests
Ignore-AOSP-First: ART Services.
Change-Id: I7642ea50180f95c61e874d069cecab2a63a2dbbe

libartservice/service/java/com/android/server/art/ArtManagerLocal.java[diff]
libartservice/service/java/com/android/server/art/ArtShellCommand.java[diff]
libartservice/service/java/com/android/server/art/BackgroundDexOptJob.java[diff]
libartservice/service/javatests/com/android/server/art/ArtManagerLocalTest.java[diff]
libartservice/service/javatests/com/android/server/art/DexOptHelperTest.java[diff]
libartservice/service/javatests/com/android/server/art/PrimaryDexOptimizerTest.java[diff]
libartservice/service/javatests/com/android/server/art/UtilsTest.java[diff]

7 files changed