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

Unified Diff: content/browser/renderer_host/media/media_stream_dispatcher_host.cc

Issue 17508005: Minor cleanup to remove the static thread-safe methods on BrowserMainLoop. Initially I thought this… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: nits Created 7 years, 6 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
Index: content/browser/renderer_host/media/media_stream_dispatcher_host.cc
===================================================================
--- content/browser/renderer_host/media/media_stream_dispatcher_host.cc (revision 207505)
+++ content/browser/renderer_host/media/media_stream_dispatcher_host.cc (working copy)
@@ -23,8 +23,11 @@
int page_request_id;
};
-MediaStreamDispatcherHost::MediaStreamDispatcherHost(int render_process_id)
- : render_process_id_(render_process_id) {
+MediaStreamDispatcherHost::MediaStreamDispatcherHost(
+ int render_process_id,
+ MediaStreamManager* media_stream_manager)
+ : render_process_id_(render_process_id),
+ media_stream_manager_(media_stream_manager) {
}
void MediaStreamDispatcherHost::StreamGenerated(
@@ -116,7 +119,7 @@
it != streams_.end();
++it) {
std::string label = it->first;
- GetManager()->StopGeneratedStream(label);
+ media_stream_manager_->StopGeneratedStream(label);
}
// Clear the map after we have stopped all the streams.
streams_.clear();
@@ -139,7 +142,7 @@
<< " ], "
<< security_origin.spec() << ")";
- const std::string& label = GetManager()->GenerateStream(
+ const std::string& label = media_stream_manager_->GenerateStream(
this, render_process_id_, render_view_id, components, security_origin);
if (label.empty()) {
Send(new MediaStreamMsg_StreamGenerationFailed(render_view_id,
@@ -158,7 +161,7 @@
for (StreamMap::iterator it = streams_.begin(); it != streams_.end(); ++it) {
if (it->second.render_view_id == render_view_id &&
it->second.page_request_id == page_request_id) {
- GetManager()->CancelRequest(it->first);
+ media_stream_manager_->CancelRequest(it->first);
}
}
}
@@ -172,7 +175,7 @@
if (it == streams_.end())
return;
- GetManager()->StopGeneratedStream(label);
+ media_stream_manager_->StopGeneratedStream(label);
streams_.erase(it);
}
@@ -187,7 +190,7 @@
<< type << ", "
<< security_origin.spec() << ")";
- const std::string& label = GetManager()->EnumerateDevices(
+ const std::string& label = media_stream_manager_->EnumerateDevices(
this, render_process_id_, render_view_id, type, security_origin);
DCHECK(!label.empty());
streams_[label] = StreamRequest(render_view_id, page_request_id);
@@ -206,15 +209,11 @@
<< type << ", "
<< security_origin.spec() << ")";
- const std::string& label = GetManager()->OpenDevice(
+ const std::string& label = media_stream_manager_->OpenDevice(
this, render_process_id_, render_view_id,
device_id, type, security_origin);
DCHECK(!label.empty());
streams_[label] = StreamRequest(render_view_id, page_request_id);
}
-MediaStreamManager* MediaStreamDispatcherHost::GetManager() {
- return BrowserMainLoop::GetMediaStreamManager();
-}
-
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698