diff options
| author | 2023-12-20 12:14:27 -0500 | |
|---|---|---|
| committer | 2023-12-20 12:38:47 -0500 | |
| commit | f9027a5955112c1162be3da5bff66d6fefcae3f2 (patch) | |
| tree | 0a88ada1d13f530fa9e7b129adcfacd1cabe7b8f /api/api.go | |
| parent | ff50b586a4b987d30f11922032cd113811189ccc (diff) | |
[Mobile] Don't return StateFlow from method
MobileConnectionsRepositoryImpl#subscriptionModelForSubId was creating a
StateFlow for each subscription that it returned. This is an incorrect
usage since that `stateIn` call starts a new job to collect from the
underlying flow, and it is never canceled until the scope is canceled.
The scope being used is the @Application scope, so it's effectively
never canceled.
It seems that this was not a load bearing StateFlow, so this CL just
removes the `stateIn` call to fix that issue.
Test: tests in statusbar/pipeline/mobile
Bug: 317205264
Flag: NONE
Change-Id: I82ddc7576e2b8d45574be6f0c3f0176da9457bb9
Diffstat (limited to 'api/api.go')
0 files changed, 0 insertions, 0 deletions