Web MIDI: add RequestMIDISysExPermission to BrowserContext

content/shell/shell_browser_context.cc

Index: content/shell/shell_browser_context.cc
diff --git a/content/shell/shell_browser_context.cc b/content/shell/shell_browser_context.cc
index 721d7c88210d7fa46c57022095f01dad747f356d..ab44f0dd11d4c82515f68bf0e519ee6be9554985 100644
--- a/content/shell/shell_browser_context.cc
+++ b/content/shell/shell_browser_context.cc
@@ -170,6 +170,15 @@ net::URLRequestContextGetter*
   return GetRequestContext();
 }
 
+void ShellBrowserContext::RequestMIDISysExPermission(
+      int render_process_id,
+      int render_view_id,
+      const GURL& requesting_frame,
+      const MIDISysExPermissionCallback& callback) {
+  // Always reject requests for testing.
+  callback.Run(false);

[jam, 2013/07/12 16:06:35]

this means that someone running content_shell won't see this feature. in
general, we try to always enable stuff in content_shell and ignore asking the
user for permission, so this should be callback.Run(true). if for layout tests
you want different behavior, you can set that programmatically.

[Takashi Toyoshima, 2013/07/12 16:37:30]

Currently layout test assumes content_shell always reject SysEx permission
request to verify such a request causes SecurityError. In the future, we'll make
this can be set programmatically later.

[jam, 2013/07/17 00:00:13]

content_shell is used as both a small browser to try out content & blink, and
also for running layout tests. so you shouldn't disable stuff by default unless
it's running in layout test mode, i.e. change this to

callback.Run(!CommandLine::ForCurrentProcess()->HasSwitch(switches::kDumpRenderTree));

+}
+
 net::URLRequestContextGetter*
     ShellBrowserContext::CreateRequestContextForStoragePartition(
         const base::FilePath& partition_path,