Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4855)

Unified Diff: chrome/browser/media/midi_permission_context.cc

Issue 2897833002: Add a PermissionContext subclass for the midi permission (Closed)
Patch Set: Midi sysex Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/media/midi_permission_context.cc
diff --git a/chrome/browser/media/midi_permission_context.cc b/chrome/browser/media/midi_permission_context.cc
index 218f9b9599c5df2e68f07bc196972c0d765600f4..7e880138b8f728f289a41fa7a529e0b94fd6008c 100644
--- a/chrome/browser/media/midi_permission_context.cc
+++ b/chrome/browser/media/midi_permission_context.cc
@@ -4,35 +4,17 @@
#include "chrome/browser/media/midi_permission_context.h"
-#include "chrome/browser/content_settings/tab_specific_content_settings.h"
-#include "chrome/browser/permissions/permission_request_id.h"
-#include "content/public/browser/child_process_security_policy.h"
-#include "url/gurl.h"
-
MidiPermissionContext::MidiPermissionContext(Profile* profile)
- : PermissionContextBase(profile,
- CONTENT_SETTINGS_TYPE_MIDI_SYSEX) {}
+ : PermissionContextBase(profile, CONTENT_SETTINGS_TYPE_MIDI) {}
MidiPermissionContext::~MidiPermissionContext() {
}
-void MidiPermissionContext::UpdateTabContext(const PermissionRequestID& id,
- const GURL& requesting_frame,
- bool allowed) {
- TabSpecificContentSettings* content_settings =
- TabSpecificContentSettings::GetForFrame(id.render_process_id(),
- id.render_frame_id());
- if (!content_settings)
- return;
-
- if (allowed) {
- content_settings->OnMidiSysExAccessed(requesting_frame);
-
- content::ChildProcessSecurityPolicy::GetInstance()->
- GrantSendMidiSysExMessage(id.render_process_id());
- } else {
- content_settings->OnMidiSysExAccessBlocked(requesting_frame);
- }
+ContentSetting MidiPermissionContext::GetPermissionStatusInternal(
+ content::RenderFrameHost* render_frame_host,
+ const GURL& requesting_origin,
+ const GURL& embedding_origin) const {
+ return CONTENT_SETTING_ALLOW;
}
bool MidiPermissionContext::IsRestrictedToSecureOrigins() const {
« no previous file with comments | « chrome/browser/media/midi_permission_context.h ('k') | chrome/browser/media/midi_permission_context_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698