add "always allow" option to the mediastream infobar and allow user to allow/not allow acces to devi

chrome/browser/ui/media_stream_infobar_delegate.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_MEDIA_STREAM_INFOBAR_DELEGATE_H_
 #define CHROME_BROWSER_UI_MEDIA_STREAM_INFOBAR_DELEGATE_H_
 
 #include <string>
 
-#include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/infobars/infobar_delegate.h"
-#include "content/public/browser/content_browser_client.h"
-#include "content/public/common/media_stream_request.h"
+#include "chrome/browser/media/media_stream_devices_controller.h"
 
-class MessageLoop;
 class TabContents;
 typedef TabContents TabContentsWrapper;
 
 // This class configures an infobar shown when a page requests access to a
 // user's microphone and/or video camera.  The user is shown a message asking
 // which audio and/or video devices he wishes to use with the current page, and
 // buttons to give access to the selected devices to the page, or to deny access
 // to them.
 class MediaStreamInfoBarDelegate : public InfoBarDelegate {
  public:
   MediaStreamInfoBarDelegate(
       InfoBarTabHelper* tab_helper,
-      const content::MediaStreamRequest* request,
-      const content::MediaResponseCallback& callback);
+      MediaStreamDevicesController* controller);
 
   virtual ~MediaStreamInfoBarDelegate();
 
   // These tell whether the user has to select audio and/or video devices.
-  bool has_audio() const { return has_audio_; }
-  bool has_video() const { return has_video_; }
+  bool has_audio() const;
+  bool has_video() const;

[Ivan Korotkov, 2012/06/14 17:14:12]

If it's easy to rename them to HasVideo/Audio, please do so.

[no longer working on chromium, 2012/06/15 16:52:07]

Done.

 
   // Returns lists of audio and/or video devices from which the user will have
   // to choose.
   content::MediaStreamDevices GetAudioDevices() const;
   content::MediaStreamDevices GetVideoDevices() const;
 
   // Returns the security origin (e.g. "www.html5rocks.com") at the origin
   // of this request.
   const GURL& GetSecurityOrigin() const;
 
+
   // Callbacks to handle accepting devices or denying the request. |audio_id|
   // and |video_id| are the device IDs of the accepted audio and video devices.
+  // |always_allow| is true if the "always allow" option is checked.
   // The |audio_id| or |video_id| values are ignored if the request did not ask
   // for audio or video devices respectively.
-  void Accept(const std::string& audio_id, const std::string& video_id);
+  void Accept(const std::string& audio_id, const std::string& video_id,

[Ivan Korotkov, 2012/06/14 17:14:12]

Should be one line per argument.

[no longer working on chromium, 2012/06/15 16:52:07]

Done.

+              bool always_allow);
   void Deny();
 
  private:
-  // Finds a device in the current request with the specified |id| and |type|,
-  // and adds it to the |devices| array.
-  void AddDeviceWithId(content::MediaStreamDeviceType type,
-                       const std::string& id,
-                       content::MediaStreamDevices* devices);
-
   // InfoBarDelegate:
   virtual InfoBar* CreateInfoBar(InfoBarTabHelper* owner) OVERRIDE;
   virtual void InfoBarDismissed() OVERRIDE;
   virtual gfx::Image* GetIcon() const OVERRIDE;
   virtual Type GetInfoBarType() const OVERRIDE;
   virtual MediaStreamInfoBarDelegate* AsMediaStreamInfoBarDelegate() OVERRIDE;
 
-  // The original request for access to devices.
-  const content::MediaStreamRequest* request_;
 
-  // The callback that needs to be Run to notify WebRTC of whether access to
-  // audio/video devices was granted or not.
-  content::MediaResponseCallback callback_;
-
-  // Whether the request is for audio and/or video devices.
-  bool has_audio_;
-  bool has_video_;
+ private:
+  scoped_ptr<MediaStreamDevicesController> controller_;
 
   DISALLOW_COPY_AND_ASSIGN(MediaStreamInfoBarDelegate);
 };
 
 #endif  // CHROME_BROWSER_UI_MEDIA_STREAM_INFOBAR_DELEGATE_H_