Update Sending Settings Changed Notifications

This CL updates piping settings changed notifications from the
ContextHub HAL to CHRE by:
- Supporting multi-user location settings by sending a notification
on a user change
- Sending a microphone access notification on a ContextHub HAL reset

Bug: 183218234
Test: Test prerequisites -  have 2 users (Owner, Guest) on device,
default location settings on owner is enabled.
- Change location setting to 'disabled' on Owner, observe that the
settings change is propagated (via a log msg).
- Change user to Guest, observe a location enabled notification
(via a log msg)
- Permutations of the above, observe correct behavior via log msgs.
- Repeat on a different device for a consistency check

Change-Id: Ib2bc7cac37ea4a0e0acdf0930df3a926b0efb1c2
This commit is contained in:
karthik bharadwaj
2021-03-22 12:25:18 -07:00
parent f9bb86f165
commit 21a6f2d5db

View File

@@ -691,6 +691,7 @@ public class ContextHubService extends IContextHubService.Stub {
sendLocationSettingUpdate();
sendWifiSettingUpdate(true /* forceUpdate */);
sendAirplaneModeSettingUpdate();
sendMicrophoneDisableSettingUpdateForCurrentUser();
mTransactionManager.onHubReset();
queryNanoAppsInternal(contextHubId);
@@ -1123,6 +1124,7 @@ public class ContextHubService extends IContextHubService.Stub {
*/
public void onUserChanged() {
Log.d(TAG, "User changed to id: " + getCurrentUserId());
sendLocationSettingUpdate();
sendMicrophoneDisableSettingUpdateForCurrentUser();
}
}