GNSS_NI_RESPONSE_IGNORE handling

GNSS_NI_RESPONSE_IGNORE inaverdently mapped to NO_RESPONSE,
which is a different reponse code. The two should be handled
differently.

Bug: 111770241
CRs-Fixed: 2296845

Change-Id: If2697f51c07a0272841709bfca247daaa4e963b6
This commit is contained in:
Kevin Tang 2018-08-14 16:38:57 -07:00
parent 329de0d44f
commit a030bf9baa

View file

@ -198,17 +198,22 @@ void GnssAPIClient::gnssNiRespond(int32_t notifId,
IGnssNiCallback::GnssUserResponseType userResponse) IGnssNiCallback::GnssUserResponseType userResponse)
{ {
LOC_LOGD("%s]: (%d %d)", __FUNCTION__, notifId, static_cast<int>(userResponse)); LOC_LOGD("%s]: (%d %d)", __FUNCTION__, notifId, static_cast<int>(userResponse));
GnssNiResponse data = GNSS_NI_RESPONSE_IGNORE; GnssNiResponse data;
if (userResponse == IGnssNiCallback::GnssUserResponseType::RESPONSE_ACCEPT) switch (userResponse) {
case IGnssNiCallback::GnssUserResponseType::RESPONSE_ACCEPT:
data = GNSS_NI_RESPONSE_ACCEPT; data = GNSS_NI_RESPONSE_ACCEPT;
else if (userResponse == IGnssNiCallback::GnssUserResponseType::RESPONSE_DENY) break;
case IGnssNiCallback::GnssUserResponseType::RESPONSE_DENY:
data = GNSS_NI_RESPONSE_DENY; data = GNSS_NI_RESPONSE_DENY;
else if (userResponse == IGnssNiCallback::GnssUserResponseType::RESPONSE_NORESP) break;
case IGnssNiCallback::GnssUserResponseType::RESPONSE_NORESP:
data = GNSS_NI_RESPONSE_NO_RESPONSE; data = GNSS_NI_RESPONSE_NO_RESPONSE;
else { break;
LOC_LOGD("%s]: invalid GnssUserResponseType: %d", __FUNCTION__, (int)userResponse); default:
return; data = GNSS_NI_RESPONSE_IGNORE;
break;
} }
locAPIGnssNiResponse(notifId, data); locAPIGnssNiResponse(notifId, data);
} }