tree 5728414fec73c30b0ac694552a5536c93bfff288
parent 753d82894de8514522dfdf490f5a4693aef65fa7
author Steven Bennetts <stevenjb@chromium.org> 1614885137 -0800
committer Commit Bot <commit-bot@chromium.org> 1618587483 +0000

shill: Cellular: Fix Create/DestroyService(s)

This moves all creation and destruction of CellularService
instances to Cellular::CreateServices or DestroyServices
and updates CellularServiceProvider accordingly.

This also changes CellularServiceProvider::RemoveServices
to actually remove the services to address the associated bug.

BUG=b:181680557
TEST=unit tests and existing tast/tauto tests pass

Change-Id: I5a49b4f01b54a693aa37862c4416f6588f9dcea8
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2737805
Tested-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Commit-Queue: Steven Bennetts <stevenjb@chromium.org>
(cherry picked from commit 4c19c92943e90bb2e525de02248f0fb7985ec16d)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2830419
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>