Web MIDI: add RequestMIDISysExPermission to BrowserContext

content/shell/shell_browser_context.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/shell/shell_browser_context.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/environment.h"
 #include "base/file_util.h"
@@ skipping 152 matching lines @@
   return GetRequestContext();
 }
 
 net::URLRequestContextGetter*
     ShellBrowserContext::GetMediaRequestContextForStoragePartition(
         const base::FilePath& partition_path,
         bool in_memory) {
   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,
         bool in_memory,
         ProtocolHandlerMap* protocol_handlers) {
   return NULL;
 }
 
 ResourceContext* ShellBrowserContext::GetResourceContext()  {
   return resource_context_.get();
 }
 
 GeolocationPermissionContext*
     ShellBrowserContext::GetGeolocationPermissionContext()  {
   return NULL;
 }
 
 quota::SpecialStoragePolicy* ShellBrowserContext::GetSpecialStoragePolicy() {
   return NULL;
 }
 
 }  // namespace content