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

content/browser/speech/speech_recognition_manager_impl.cc

Index: content/browser/speech/speech_recognition_manager_impl.cc
diff --git a/content/browser/speech/speech_recognition_manager_impl.cc b/content/browser/speech/speech_recognition_manager_impl.cc
index 4b0827a6b62bb53544695c310d57f048e15bc2e3..adadd3a1e57d74847a5278ad8c361eb2ed531a55 100644
--- a/content/browser/speech/speech_recognition_manager_impl.cc
+++ b/content/browser/speech/speech_recognition_manager_impl.cc
@@ -194,9 +194,10 @@ void SpeechRecognitionManagerImpl::RecognitionAllowedCallback(int session_id,
 
   if (ask_user) {
     SpeechRecognitionSessionContext& context = session->context;
+    DCHECK_NE(context.render_frame_id, 0);

[ncarter (slow), 2014/07/10 01:17:52]

Do you want to check "!= 0" or do you want to check "> 0"? Are you okay if it's
MSG_ROUTING_NONE?

[miu, 2014/07/10 22:16:12]

My intention was to confirm the value was being set, but I think I should
probably remove this DCHECK and guard the whole call to MakeMediaAccessRequest()
with a sanity-check.  In other words, we shouldn't be making the request unless
the RP ID and RF ID are both valid.

     context.label = media_stream_manager_->MakeMediaAccessRequest(
         context.render_process_id,
-        context.render_view_id,
+        context.render_frame_id,
         context.request_id,
         StreamOptions(true, false),
         GURL(context.context_name),