ContentBrowserClient::RequestPermission replies with PermissionStatus instead of bool.

content/browser/permissions/permission_service_impl.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/permissions/permission_service_impl.h"
 
 #include "content/public/browser/content_browser_client.h"
 
 namespace content {
 
@@ skipping 67 matching lines @@
       user_gesture, // TODO(mlamouri): should be removed (crbug.com/423770)
       base::Bind(&PermissionServiceImpl::OnRequestPermissionResponse,
                  weak_factory_.GetWeakPtr(),
                  callback,
                  request_id));
 }
 
 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() {
   DCHECK(context_->web_contents());
 
   for (RequestsMap::Iterator<PendingRequest> it(&pending_requests_);
        !it.IsAtEnd(); it.Advance()) {
     GetContentClient()->browser()->CancelPermissionRequest(
         it.GetCurrentValue()->permission,
         context_->web_contents(),
@@ skipping 46 matching lines @@
 void PermissionServiceImpl::ResetPermissionStatus(PermissionType type,
                                                   GURL origin) {
   // If the embedding_origin is empty we'll use |origin| instead.
   GURL embedding_origin = context_->GetEmbeddingOrigin();
   GetContentClient()->browser()->ResetPermission(
       type, context_->GetBrowserContext(), origin,
       embedding_origin.is_empty() ? origin : embedding_origin);
 }
 
 }  // namespace content