Group the different permission related methods in the content api.

content/browser/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/media/midi_dispatcher_host.h"
 
 #include "base/bind.h"
 #include "content/browser/child_process_security_policy_impl.h"
 #include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/common/media/midi_messages.h"
@@ skipping 60 matching lines @@
     int bridge_id,
     const GURL& origin,
     bool user_gesture) {
   int render_process_id = render_frame_host->GetProcess()->GetID();
   int render_frame_id = render_frame_host->GetRoutingID();
 
   PendingPermission pending_permission(
       render_process_id, render_frame_id, bridge_id);
   pending_permissions_.push_back(pending_permission);
 
-  GetContentClient()->browser()->RequestMidiSysExPermission(
+  GetContentClient()->browser()->RequestPermission(
+      content::PERMISSION_MIDI_SYSEX,
       web_contents(),
       bridge_id,
       origin,
       user_gesture,
       base::Bind(&MidiDispatcherHost::WasSysExPermissionGranted,
                  weak_factory_.GetWeakPtr(),
-                 render_process_id, render_frame_id, bridge_id),
-      &pending_permissions_.back().cancel);
+                 render_process_id, render_frame_id, bridge_id));
 }
 
 void MidiDispatcherHost::CancelPermissionRequestsForFrame(
     RenderFrameHost* render_frame_host) {
   int render_process_id = render_frame_host->GetProcess()->GetID();
   int render_frame_id = render_frame_host->GetRoutingID();
 
   for (size_t i = 0; i < pending_permissions_.size(); ++i) {
     if (pending_permissions_[i].render_process_id == render_process_id &&
         pending_permissions_[i].render_frame_id == render_frame_id) {
-      if (!pending_permissions_[i].cancel.is_null())
-        pending_permissions_[i].cancel.Run();
+      GetContentClient()->browser()->CancelPermissionRequest(
+          content::PERMISSION_MIDI_SYSEX,

[jam, 2014/10/20 20:13:57]

nit: here and above, and elsewhere, no "content::"

[Miguel Garcia, 2014/10/21 17:17:12]

Done.

+          web_contents(), pending_permissions_[i].bridge_id,
+          render_frame_host->GetLastCommittedURL());
+
       pending_permissions_.erase(pending_permissions_.begin() + i);
       return;
     }
   }
 }
 
 void MidiDispatcherHost::WasSysExPermissionGranted(int render_process_id,
                                                    int render_frame_id,
                                                    int bridge_id,
                                                    bool is_allowed) {
@@ skipping 13 matching lines @@
             GrantSendMidiSysExMessage(render_process_id);
       }
 
       pending_permissions_.erase(pending_permissions_.begin() + i);
       return;
     }
   }
 }
 
 }  // namespace content