| Index: content/browser/permissions/permission_service_impl.cc
|
| diff --git a/content/browser/permissions/permission_service_impl.cc b/content/browser/permissions/permission_service_impl.cc
|
| index bd5c87b8223a79c02c8a71a71adc42e7e8945733..a086c79ea1a15b8d36c2544e2a1756280b99ec8a 100644
|
| --- a/content/browser/permissions/permission_service_impl.cc
|
| +++ b/content/browser/permissions/permission_service_impl.cc
|
| @@ -85,12 +85,14 @@ void PermissionServiceImpl::RequestPermission(
|
| void PermissionServiceImpl::OnRequestPermissionResponse(
|
| const mojo::Callback<void(PermissionStatus)>& callback,
|
| int request_id,
|
| - bool allowed) {
|
| + PermissionStatus status) {
|
| pending_requests_.Remove(request_id);
|
|
|
| - // TODO(mlamouri): for now, we only get a boolean back, but we would ideally
|
| - // need a ContentSetting, see http://crbug.com/432978
|
| - callback.Run(allowed ? PERMISSION_STATUS_GRANTED : PERMISSION_STATUS_ASK);
|
| + // TODO(mlamouri): this is not yet returning a tri-state value because it will
|
| + // require some further changes that will be dealt with in
|
| + // https://crrev.com/794203004/
|
| + status = status == PERMISSION_STATUS_DENIED ? PERMISSION_STATUS_ASK : status;
|
| + callback.Run(status);
|
| }
|
|
|
| void PermissionServiceImpl::CancelPendingRequests() {
|
|
|