Remove the usages of WorkManager's SettableFuture

[kuanyingchou]

Problem description

Hi, we would like to remove SettableFuture from WorkManager but found that LeakCanary is using it:

/~https://github.com/search?q=repo%3Asquare%2Fleakcanary%20settablefuture&type=code

Potential solutions

Is it possible to remove the usages of SettableFuture as it's WorkManager's internal API?

Additional information

Here's our change in WorkManager: https://android-review.googlesource.com/c/platform/frameworks/support/+/3000016/1

[pyricau]

@kuanyingchou Do you have a suggestion on what to replace it with? I don't mind updating, just need some guidance on how to implement getForegroundInfoAsync(): ListenableFuture<ForegroundInfo>. I suspect I did this based on some example implementation I found, no idea where though.

That being said, even after updating, it'll still be in older releases of LeakCanary.

I'm also surprised because that class has @RestrictTo(RestrictTo.Scope.LIBRARY_GROUP), I wonder why the compiler didn't yell at me.

[pyricau]

Conversation in ASG for anyone who has access: https://androidstudygroup.slack.com/archives/CAYMXJE0H/p1713384243471319

[kuanyingchou]

Hi, @pyricau , thanks for the reply! I think it can be replaced with getFuture() or launchFuture() (for suspend functions) from androidx.concurrent (some examples here and here), but it may introduce a new dependency.

I'm not sure about the update path either. Let's see what the folks on ASG say.