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

chrome/browser/media/chrome_midi_permission_context.h

 // 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.
 
 #ifndef CHROME_BROWSER_MEDIA_CHROME_MIDI_PERMISSION_CONTEXT_H_
 #define CHROME_BROWSER_MEDIA_CHROME_MIDI_PERMISSION_CONTEXT_H_
 
 #include "base/memory/scoped_ptr.h"
 #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;
 class Profile;
 
 // This class manages MIDI permissions flow. Used on the UI thread.
 class ChromeMidiPermissionContext : public BrowserContextKeyedService {
  public:
   explicit ChromeMidiPermissionContext(Profile* profile);
   virtual ~ChromeMidiPermissionContext();
 
   // BrowserContextKeyedService methods:
   virtual void Shutdown() OVERRIDE;
 
   // Request to ask users permission about MIDI.
   void RequestMidiSysExPermission(
       int render_process_id,
       int render_view_id,
       int bridge_id,
       const GURL& requesting_frame,
       const content::BrowserContext::MidiSysExPermissionCallback& callback);
 
   // Cancel a pending MIDI permission request.
   void CancelMidiSysExPermissionRequest(int render_process_id,
                                         int render_view_id,
                                         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,
       const content::BrowserContext::MidiSysExPermissionCallback& callback);
 
   // Called when permission is granted without interactively asking the user.
   void PermissionDecided(
       const PermissionRequestID& id,
       const GURL& requesting_frame,
       const GURL& embedder,
       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();
 
   // Removes any pending InfoBar request.
   void CancelPendingInfobarRequest(const PermissionRequestID& id);
 
   Profile* const profile_;
   bool shutting_down_;
   scoped_ptr<PermissionQueueController> permission_queue_controller_;
 
   DISALLOW_COPY_AND_ASSIGN(ChromeMidiPermissionContext);
 };
 
 #endif  // CHROME_BROWSER_MEDIA_CHROME_MIDI_PERMISSION_CONTEXT_H_