Commit 4f8538cc authored by Takashi Toyoshima's avatar Takashi Toyoshima Committed by Commit Bot

Web MIDI: WebView ChildProcessSecurityPolicy support

PermissionManager needs to update ChildProcessSecurityPolicy
for MIDI_SYSEX and WebView implementation, AwPermissionManager
didn't.

This patch adds ChildProcessSecurityPolicy support to
AwPermissionManager.

Bug: b/123485223, 987505
Change-Id: Ibf6592b7b69780d11361a03700c19fd08a028bab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1476930
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Reviewed-by: default avatarRichard Coles <torne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#680741}
parent 21db0fea
...@@ -13,6 +13,7 @@ ...@@ -13,6 +13,7 @@
#include "base/bind.h" #include "base/bind.h"
#include "base/callback.h" #include "base/callback.h"
#include "base/logging.h" #include "base/logging.h"
#include "content/public/browser/child_process_security_policy.h"
#include "content/public/browser/permission_controller.h" #include "content/public/browser/permission_controller.h"
#include "content/public/browser/permission_type.h" #include "content/public/browser/permission_type.h"
#include "content/public/browser/render_frame_host.h" #include "content/public/browser/render_frame_host.h"
...@@ -179,6 +180,11 @@ class AwPermissionManager::PendingRequest { ...@@ -179,6 +180,11 @@ class AwPermissionManager::PendingRequest {
} }
DCHECK(!IsCompleted()); DCHECK(!IsCompleted());
results[result->second] = status; results[result->second] = status;
if (type == PermissionType::MIDI_SYSEX &&
status == PermissionStatus::GRANTED) {
content::ChildProcessSecurityPolicy::GetInstance()
->GrantSendMidiSysExMessage(render_process_id);
}
resolved_permissions_.insert(type); resolved_permissions_.insert(type);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment