Desktop chrome tab capture-chooseDesktopMedia()

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 bd21240cdc56f3117fc2fe2f0a1e2052ef5cd7cd..eed4c513b5493de9d9d545e9d3da9bc8ee05e676 100644
--- a/chrome/browser/media/desktop_streams_registry.cc
+++ b/chrome/browser/media/desktop_streams_registry.cc
@@ -39,6 +39,11 @@ std::string DesktopStreamsRegistry::RegisterStream(
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
   std::string id = GenerateRandomStreamId();
+
+  // Give the ability of identify tab type from id.
+  if (source.type == content::DesktopMediaID::TYPE_TAB)
+    id = source.ToString();

[miu, 2015/12/08 01:54:38]

This is a security violation.  If you need this information, please add it to
the ApprovedDesktopMediaStream struct.

[GeorgeZ, 2015/12/09 19:36:37]

I will remove this one. I remembered this change is not definite needed. It is
just better to have. if I am wrong, I will figure it out in the next CL that is
going to hook up webcontentscapture.

+
   DCHECK(approved_streams_.find(id) == approved_streams_.end());
   ApprovedDesktopMediaStream& stream = approved_streams_[id];
   stream.render_process_id = render_process_id;