Activity Leak in PermissionsControllerImpl during Configuration Change Fix

[0ximDigital]

This PR is a proposed fix for the issue #181

Proposed Solution

The following changes are made to the PermissionsControllerImpl:

1. Using MutableSharedFlow to suspend while waiting for the result to emit permission request results instead of using suspendCoroutine
2. Unregistering launcher holder if there is any, during the activity destroy lifecycle callback

Open to discussion:

At the start of the bind() method, there is an unbindActivity() method invocation to unbind previously bound Activity if developers mistakenly invoke the bind() method more times for different Activities - Android MultiWindow?

We could throw an exception there instead to signal to developers that they may be making a mistake instead.

Most Important

These changes do not change a behaviour of the PermissionControllerImpl or the API in any way to keep it backwards compatible.