Begin adding support for tab mirroring via the MediaStream audio/video capturing

chrome/browser/ui/media_stream_infobar_delegate.h

Index: chrome/browser/ui/media_stream_infobar_delegate.h
diff --git a/chrome/browser/ui/media_stream_infobar_delegate.h b/chrome/browser/ui/media_stream_infobar_delegate.h
index 8e66367a7a2a651ce7d3c662fd8e12e04aa48125..2debe5a98b22266b774db7011856f7cd6c10bca2 100644
--- a/chrome/browser/ui/media_stream_infobar_delegate.h
+++ b/chrome/browser/ui/media_stream_infobar_delegate.h
@@ -39,7 +39,7 @@ class MediaStreamInfoBarDelegate : public InfoBarDelegate {
 
   // Returns the security origin (e.g. "www.html5rocks.com") at the origin
   // of this request.
-  const GURL& GetSecurityOrigin() const;
+  const std::string& GetSecurityOriginSpec() const;

[tommi (sloooow) - chröme, 2012/09/10 09:17:25]

Why this change?  Seems to me that a full GURL is more useful.

[miu, 2012/09/10 21:24:38]

xians@ addressed this (in response to a comment in
media_stream_devices_controller.cc).

I'll add that there's a purity aspect here as well: Meaning, that UI code
shouldn't include logic that decides how to use some "internal" object. 
Instead, it's the responsibility of the model to provide the string the UI
should show.

On 2012/09/10 09:17:25, tommi wrote:

 
   void set_selected_audio_device(const std::string& device_id) {
     selected_audio_device_ = device_id;
@@ -57,6 +57,11 @@ class MediaStreamInfoBarDelegate : public InfoBarDelegate {
   }
   bool always_allow() const { return always_allow_; }
 
+  // These determine whether all audio (or video) devices can be auto-accepted
+  // in the future should the user accept them this time.
+  bool IsSafeToAlwaysAllowAudio() const;
+  bool IsSafeToAlwaysAllowVideo() const;
+
   // Callbacks to handle accepting devices or denying the request.
   void Accept();
   void Deny();