Removed the use of WebFrame::frameForCurrentContext() in MediaStreamCenter::didStopLocalMediaStream

content/renderer/media/media_stream_center.cc

Index: content/renderer/media/media_stream_center.cc
diff --git a/content/renderer/media/media_stream_center.cc b/content/renderer/media/media_stream_center.cc
index b332c1334896a7d9c3a547d1b1dae5ba0e922939..8af49135cd5de8ca7fb0930f16003f2bb5f770ee 100644
--- a/content/renderer/media/media_stream_center.cc
+++ b/content/renderer/media/media_stream_center.cc
@@ -27,15 +27,6 @@
 
 namespace content {
 
-static MediaStreamImpl* GetMediaStreamImpl(WebKit::WebFrame* web_frame) {
-  RenderViewImpl* render_view = RenderViewImpl::FromWebView(web_frame->view());
-  if (!render_view)
-    return NULL;
-
-  // TODO(perkj): Avoid this cast?
-  return static_cast<MediaStreamImpl*>(render_view->userMediaClient());
-}
-
 static webrtc::MediaStreamInterface* GetNativeMediaStream(
     const WebKit::WebMediaStreamDescriptor& stream) {
   MediaStreamExtraData* extra_data =
@@ -116,16 +107,9 @@ void MediaStreamCenter::didDisableMediaStreamTrack(
 void MediaStreamCenter::didStopLocalMediaStream(
     const WebKit::WebMediaStreamDescriptor& stream) {
   DVLOG(1) << "MediaStreamCenter::didStopLocalMediaStream";
-  WebKit::WebFrame* web_frame = WebKit::WebFrame::frameForCurrentContext();
-  if (!web_frame)
-    return;
-  MediaStreamImpl* ms_impl = GetMediaStreamImpl(web_frame);
-  if (ms_impl) {
-    ms_impl->StopLocalMediaStream(stream);
-    return;
-  }
-
-  NOTREACHED();
+  MediaStreamExtraData* extra_data =
+       static_cast<MediaStreamExtraData*>(stream.extraData());
+  return extra_data->RunStreamStopCallback();

[tommi (sloooow) - chröme, 2012/09/21 09:18:03]

no need for return statements in void functions

[perkj_chrome, 2012/09/21 10:47:10]

Done.

 }
 
 void MediaStreamCenter::didCreateMediaStream(