Prevent incorrect background processing

[kmicha19-ford]

Fixes #1799 (when used in conjunction with EASession left open PR #kmicha19-ford#1)

This PR is ready for review.

Risk

This PR makes no API changes.

Testing Plan

• I have verified that I have not introduced new warnings in this PR (or explain why below)
• I have tested this PR against Core and verified behavior.

Unit Tests

• Does not affect any unit tests.

Core Tests

• Built the Obj-C example app and verified that all interactions work
  • iAP disconnect / reconnect
  • LockScreen on/off rapidly
• Built the Swift example app and verified that all interactions work
  • iAP disconnect / reconnect
  • LockScreen on/off rapidly

Summary

This fixes the bug where an app could run on and on unnecessarily even when backgrounded and disconnected.

Changelog

Breaking Changes

• None

Enhancements

• SDLifeCycleManager no longer uses the SDLBackgroundTaskManager.
• SDLBackgroundTaskManager uses the iOS instance method beginBackgroundTask(withName taskName: String?,
  expirationHandler handler: (() -> Void)? = nil) -> UIBackgroundTaskIdentifier
• beginBackgroundTask is for requesting additional processing time when an app is backgrounded and before being suspended so an app can finish tasks, such as saving data, before being suspended. This method is not meant to keep an app running in the background. After 30 seconds iOS may or may not terminate the app
• Using beginBackgroundTask in this manner may conflict with EAAccessory Connect / Disconnect app management and keep the app running in the background when it should be suspended. This may cause undo battery drain.
• It should be noted that SDLSecondaryTransportManager correctly uses SDLBackgroundTaskManager.

Tasks Remaining:

• None

CLA

• I have signed the CLA

[joeljfischer]

@makinney I apologize for not noticing earlier, but is this a duplicate of #1906? Is an additional PR intended to be submitted sometime soon as well?

[kmicha19-ford]

will recreate

[kmicha19-ford]

Hi Joel, These are new as of today. #1910 #1911 Hopefully these are correct. Let me know otherwise. Sorry for any delays. I never done any PRs on public repos before (can you tell?) Mike

…

________________________________ From: Joel Fischer <notifications@github.com> Sent: Wednesday, February 3, 2021 3:02 PM To: smartdevicelink/sdl_ios <sdl_ios@noreply.github.com> Cc: Kinney, Michael (K.A.) <kmicha19@ford.com>; Author <author@noreply.github.com> Subject: Re: [smartdevicelink/sdl_ios] Prevent incorrect background processing (#1909) @makinney<https://clicktime.symantec.com/3xK7LZuReKqWNsnx9iUUQj7Vc?u=https%3A%2F%2Fgithub.com%2Fmakinney> I apologize for not noticing earlier, but is this a duplicate of #1906<https://clicktime.symantec.com/3HmpiK8gwSVu3UcXmg7CRXd7Vc?u=https%3A%2F%2Fgithub.com%2Fsmartdevicelink%2Fsdl_ios%2Fpull%2F1906>? Is an additional PR intended to be submitted sometime soon as well? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<https://clicktime.symantec.com/3HJix5BMK6dVCYMKcuJPezc7Vc?u=https%3A%2F%2Fgithub.com%2Fsmartdevicelink%2Fsdl_ios%2Fpull%2F1909%23issuecomment-772784558>, or unsubscribe<https://clicktime.symantec.com/3LNWJGySN3iA6WG4UTCA8oi7Vc?u=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIF2K3Y6XDGFWLHXHHZWQHTS5GTXDANCNFSM4XA75RUQ>.