Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(38)

Unified Diff: content/renderer/media/media_stream_center.cc

Issue 10947030: Removed the use of WebFrame::frameForCurrentContext() in MediaStreamCenter::didStopLocalMediaStream (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Addressing code review comments found by Tommi. Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | content/renderer/media/media_stream_dependency_factory.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..c6c00766fffa5df15017bb0fcf932aee01824643 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,14 @@ 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);
+ MediaStreamExtraData* extra_data =
+ static_cast<MediaStreamExtraData*>(stream.extraData());
+ if (!extra_data) {
+ NOTREACHED();
return;
}
- NOTREACHED();
+ extra_data->OnLocalStreamStop();
}
void MediaStreamCenter::didCreateMediaStream(
« no previous file with comments | « no previous file | content/renderer/media/media_stream_dependency_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698