[MIDI] Add a permission bubble client for MIDI system control.

chrome/browser/media/chrome_midi_permission_context.h

Index: chrome/browser/media/chrome_midi_permission_context.h
diff --git a/chrome/browser/media/chrome_midi_permission_context.h b/chrome/browser/media/chrome_midi_permission_context.h
index 8b3b22affe966ac7fc7011b3fc194453c056c791..3b12df060141615cf36c70dc66af1781d193c860 100644
--- a/chrome/browser/media/chrome_midi_permission_context.h
+++ b/chrome/browser/media/chrome_midi_permission_context.h
@@ -9,6 +9,10 @@
 #include "components/browser_context_keyed_service/browser_context_keyed_service.h"
 #include "content/public/browser/browser_context.h"
 
+namespace content {
+class WebContents;
+}
+
 class GURL;
 class PermissionQueueController;
 class PermissionRequestID;
@@ -37,11 +41,20 @@ class ChromeMidiPermissionContext : public BrowserContextKeyedService {
                                         int bridge_id,
                                         const GURL& requesting_frame);
 
+  // Called when the permission decision is made. It may be by the

[Ami GONE FROM CHROMIUM, 2014/02/06 19:51:17]

s/be by/be called by/ ?

[Greg Billock, 2014/02/07 18:20:55]

reworded.

+  // InfoBarDelegate to notify permission has been set.

[Ami GONE FROM CHROMIUM, 2014/02/06 19:51:17]

I'm no mind reader but I'm pretty sure you can rewrite this comment to be either
more informative or less verbose :)
(I realize the comment predates your CL)

[Greg Billock, 2014/02/07 18:20:55]

:-) yes. clarified.

+  void NotifyPermissionSet(
+      const PermissionRequestID& id,
+      const GURL& requesting_frame,
+      const content::BrowserContext::MidiSysExPermissionCallback& callback,
+      bool allowed);
+
  private:
   // Decide whether the permission should be granted.
   // Calls PermissionDecided if permission can be decided non-interactively,
   // or NotifyPermissionSet if permission decided by presenting an infobar.
   void DecidePermission(
+      content::WebContents* web_contents,
       const PermissionRequestID& id,
       const GURL& requesting_frame,
       const GURL& embedder,
@@ -55,14 +68,6 @@ class ChromeMidiPermissionContext : public BrowserContextKeyedService {
       const content::BrowserContext::MidiSysExPermissionCallback& callback,
       bool allowed);
 
-  // Called when the permission decision is made. It may be by the
-  // InfoBarDelegate to notify permission has been set.
-  void NotifyPermissionSet(
-      const PermissionRequestID& id,
-      const GURL& requesting_frame,
-      const content::BrowserContext::MidiSysExPermissionCallback& callback,
-      bool allowed);
-
   // Return an instance of the infobar queue controller, creating it if needed.
   PermissionQueueController* GetQueueController();