Fix incorrect predicate in PermissionControllerImpl on Android

[MrPranklin]

In PR #105 permission state NotGranted is introduced and the negation when checking shouldShowRequestPermissionRationale is removed.

The desired result in that change is to return PermissionState.Denied in case that shouldShowRequestPermissionRationale returns true for all permissions, and PermissionState.NotGranted otherwise. That is correct behavior according to the documentation

However, in PR #115, possibly due to conflict resolution, the negation is brought back, while the return statement stays as it was in #105, resulting in the default state of the permission to be Denied (instead of NotGranted), which is incorrect.

[Alex009]

@MrPranklin hi. why default state should be Denied? at first run user should receive some UI for NotGranted state. but if we will have Denied - user will see "You denied access, please give us permissions".

[MrPranklin]

Hi @Alex009, in version 0.18.1 I'm getting Denied as the default state and then NotGranted after denying once, which is incorrect.

I found the reason for that to be the series of changes I described with the 2 pull requests.