Cancel MIDI permission request infobar on MIDIAccess stop.

content/browser/renderer_host/media/midi_dispatcher_host.cc

 // Copyright 2013 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/browser/renderer_host/media/midi_dispatcher_host.h"
 
 #include "base/bind.h"
 #include "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/common/media/midi_messages.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "url/gurl.h"
 
 namespace content {
 
-MIDIDispatcherHost::MIDIDispatcherHost(int render_process_id,
-                                       BrowserContext* browser_context)
+MIDIDispatcherHost::MIDIDispatcherHost(
+    int render_process_id,
+    MidiPermissionContext* midi_permission_context_)
     : render_process_id_(render_process_id),
-      browser_context_(browser_context) {
+      midi_permission_context_(midi_permission_context_) {
 }
 
 MIDIDispatcherHost::~MIDIDispatcherHost() {
 }
 
 bool MIDIDispatcherHost::OnMessageReceived(const IPC::Message& message,
                                            bool* message_was_ok) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP_EX(MIDIDispatcherHost, message, *message_was_ok)
     IPC_MESSAGE_HANDLER(MIDIHostMsg_RequestSysExPermission,
                         OnRequestSysExPermission)
+    IPC_MESSAGE_HANDLER(MIDIHostMsg_CancelSysExPermissionRequest,
+                        OnCancelSysExPermissionRequest)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP_EX()
   return handled;
 }
 
 void MIDIDispatcherHost::OverrideThreadForMessage(
     const IPC::Message& message, BrowserThread::ID* thread) {
   if (message.type() == MIDIHostMsg_RequestSysExPermission::ID)

[jam, 2013/10/24 16:15:26]

if (IPC_MESSAGE_CLASS(msg) == MIDIMsgStart)
  *thread = BrowserThread::UI;

since the midi context interface is from BrowserContext (which is UI thread),
you should also make that interface be called on the UI thread. then you don't
need thread hopping in the chrome implementation

[Kibeom Kim (inactive), 2013/10/24 19:36:26]

Done.

     *thread = BrowserThread::UI;
 }
 
 void MIDIDispatcherHost::OnRequestSysExPermission(int render_view_id,
-                                                  int client_id,
+                                                  int bridge_id,
                                                   const GURL& origin) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 
-  browser_context_->RequestMIDISysExPermission(
-      render_process_id_,
-      render_view_id,
-      origin,
-      base::Bind(&MIDIDispatcherHost::WasSysExPermissionGranted,
-                 base::Unretained(this),
-                 render_view_id,
-                 client_id));
+  if (midi_permission_context_.get()) {
+    midi_permission_context_->RequestMIDISysExPermission(
+        render_process_id_,
+        render_view_id,
+        bridge_id,
+        origin,
+        base::Bind(&MIDIDispatcherHost::WasSysExPermissionGranted,
+                   base::Unretained(this),
+                   render_view_id,
+                   bridge_id));
+  } else {
+    BrowserThread::PostTask(
+        BrowserThread::UI,
+        FROM_HERE,
+        base::Bind(&MIDIDispatcherHost::WasSysExPermissionGranted,
+                   base::Unretained(this),
+                   render_view_id,
+                   bridge_id,
+                   true));
+  }
+}
+
+void MIDIDispatcherHost::OnCancelSysExPermissionRequest(
+    int render_view_id,
+    int bridge_id,
+    const GURL& requesting_frame) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+  DVLOG(1) << __FUNCTION__ << " " << render_process_id_ << ":" << render_view_id
+           << ":" << bridge_id;
+  if (midi_permission_context_.get()) {
+    midi_permission_context_->CancelMIDISysExPermissionRequest(
+        render_process_id_, render_view_id, bridge_id, requesting_frame);
+  }
 }
 
 void MIDIDispatcherHost::WasSysExPermissionGranted(int render_view_id,
-                                                   int client_id,
+                                                   int bridge_id,
                                                    bool success) {
   ChildProcessSecurityPolicyImpl::GetInstance()->GrantSendMIDISysExMessage(
       render_process_id_);
-  Send(new MIDIMsg_SysExPermissionApproved(render_view_id, client_id, success));
+  Send(new MIDIMsg_SysExPermissionApproved(render_view_id, bridge_id, success));
 }
 
 }  // namespace content