Don't throw exception when changing unrequested permission state.

And just log it instead.

Because such an API call is always racy with package changes, especially
when someone is doing things in response to changes (e.g. the role
subsystem). We have to either catch all security exceptions at the call
site, or throw less exceptions while keeping the more important ones. So
not throwing in this case is arguably the better option.

Fixes: 321559226
Test: presubmit
Change-Id: I0835ad01af2a41b711b66b8a0d84a10c1305c401