[Cross-Site Isolation] Migrate entire MediaStream verticals to be per-RenderFrame.

chrome/browser/media/desktop_streams_registry.cc

Index: chrome/browser/media/desktop_streams_registry.cc
diff --git a/chrome/browser/media/desktop_streams_registry.cc b/chrome/browser/media/desktop_streams_registry.cc
index f2e2607d63c9b40febd89be6e5dba7baf3bd393f..85033fc9783bdccc28ee611ac9d63ddfc073b986 100644
--- a/chrome/browser/media/desktop_streams_registry.cc
+++ b/chrome/browser/media/desktop_streams_registry.cc
@@ -32,16 +32,19 @@ DesktopStreamsRegistry::~DesktopStreamsRegistry() {}
 
 std::string DesktopStreamsRegistry::RegisterStream(
     int render_process_id,
-    int render_view_id,
+    int render_frame_id,
     const GURL& origin,
     const content::DesktopMediaID& source,
     const std::string& extension_name) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
 
-  std::string id = GenerateRandomStreamId();
+  std::string id;
+  do {
+    id = GenerateRandomStreamId();
+  } while (approved_streams_.find(id) != approved_streams_.end());

[Sergey Ulanov, 2014/07/16 19:29:47]

nit: This isn't really necessary. There are 128 bits of randomness - so
probability of collision is negligible, even if you generate 1M streams here.

[miu, 2014/07/16 20:43:29]

Okay, reverted.  I did leave in a DCHECK as a sanity-check to confirm the
GenerateRandomStreamId() function isn't being non-random in debug builds.

   ApprovedDesktopMediaStream& stream = approved_streams_[id];
   stream.render_process_id = render_process_id;
-  stream.render_view_id = render_view_id;
+  stream.render_frame_id = render_frame_id;
   stream.origin = origin;
   stream.source = source;
   stream.extension_name = extension_name;
@@ -58,7 +61,7 @@ std::string DesktopStreamsRegistry::RegisterStream(
 content::DesktopMediaID DesktopStreamsRegistry::RequestMediaForStreamId(
     const std::string& id,
     int render_process_id,
-    int render_view_id,
+    int render_frame_id,
     const GURL& origin,
     std::string* extension_name) {
   DCHECK(content::BrowserThread::CurrentlyOn(content::BrowserThread::UI));
@@ -69,7 +72,7 @@ content::DesktopMediaID DesktopStreamsRegistry::RequestMediaForStreamId(
   // the same origin and the same renderer.
   if (it == approved_streams_.end() ||
       render_process_id != it->second.render_process_id ||
-      render_view_id != it->second.render_view_id ||
+      render_frame_id != it->second.render_frame_id ||
       origin != it->second.origin) {
     return content::DesktopMediaID();
   }
@@ -86,4 +89,4 @@ void DesktopStreamsRegistry::CleanupStream(const std::string& id) {
 }
 
 DesktopStreamsRegistry::ApprovedDesktopMediaStream::ApprovedDesktopMediaStream()
-    : render_process_id(-1), render_view_id(-1) {}
+    : render_process_id(-1), render_frame_id(-1) {}