Skip to content

Commit

Permalink
chore: add currently-unused should_include_device_status arg to GetPe…
Browse files Browse the repository at this point in the history
…rmissionStatusForCurrentDocument()

Xref: https://chromium-review.googlesource.com/c/chromium/src/+/5545382
  • Loading branch information
ckerr committed May 31, 2024
1 parent 3934475 commit bc35c93
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion shell/browser/electron_permission_manager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -377,7 +377,8 @@ ElectronPermissionManager::CheckProtectedUSBClasses(
blink::mojom::PermissionStatus
ElectronPermissionManager::GetPermissionStatusForCurrentDocument(
blink::PermissionType permission,
content::RenderFrameHost* render_frame_host) {
content::RenderFrameHost* render_frame_host,
bool /*should_include_device_status*/) {
if (render_frame_host->IsNestedWithinFencedFrame())
return blink::mojom::PermissionStatus::DENIED;

Expand Down
3 changes: 2 additions & 1 deletion shell/browser/electron_permission_manager.h
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,8 @@ class ElectronPermissionManager : public content::PermissionControllerDelegate {
const url::Origin& embedding_origin) override;
blink::mojom::PermissionStatus GetPermissionStatusForCurrentDocument(
blink::PermissionType permission,
content::RenderFrameHost* render_frame_host) override;
content::RenderFrameHost* render_frame_host,
bool should_include_device_status) override;
blink::mojom::PermissionStatus GetPermissionStatusForWorker(
blink::PermissionType permission,
content::RenderProcessHost* render_process_host,
Expand Down

0 comments on commit bc35c93

Please sign in to comment.