[Kiosk] Deny permission requests to other origins but the main app
Currently we were displaying permission prompts in web kiosk mode for
origins of not the main app. To keep it consistent with other types of
kiosks, we should deny those requests.
We are not explicitly denying them, but leaving them unanswered, this
preventing them from being stored into prefs.
Bug: 1044551
Change-Id: Ic6f1a260d3f03b92fc52f8f341c6259155b791d5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2014403
Commit-Queue: Balazs Engedy <[email protected]>
Reviewed-by: Balazs Engedy <[email protected]>
Cr-Commit-Position: refs/heads/master@{#734443}
diff --git a/chrome/browser/permissions/permission_request.h b/chrome/browser/permissions/permission_request.h
index 6e4db4a..b7b1ec4 100644
--- a/chrome/browser/permissions/permission_request.h
+++ b/chrome/browser/permissions/permission_request.h
@@ -131,6 +131,9 @@
// It is safe for the request to be deleted at this point -- it will receive
// no further message from the permission request system. This method will
// eventually be called on every request which is not unregistered.
+ // It is ok to call this method without actually resolving the request via
+ // PermissionGranted(), PermissionDenied() or Canceled(). However, it will not
+ // resolve the javascript promise from the requesting origin.
virtual void RequestFinished() = 0;
// Used to record UMA metrics for permission requests.
