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

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

Issue 7948004: Removing singelton property of MediaStreamManager. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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
Index: content/browser/renderer_host/media/video_capture_controller.cc
diff --git a/content/browser/renderer_host/media/video_capture_controller.cc b/content/browser/renderer_host/media/video_capture_controller.cc
index bdad7e07e377087b2e06a814ee254f3e85c7071b..82dbabbd082d85618d2fc161734cbed90e428f87 100644
--- a/content/browser/renderer_host/media/video_capture_controller.cc
+++ b/content/browser/renderer_host/media/video_capture_controller.cc
@@ -16,11 +16,13 @@ static const size_t kNoOfDIBS = 3;
VideoCaptureController::VideoCaptureController(
const VideoCaptureControllerID& id,
base::ProcessHandle render_process,
- VideoCaptureControllerEventHandler* event_handler)
+ VideoCaptureControllerEventHandler* event_handler,
+ media_stream::VideoCaptureManager* video_capture_manager)
: render_handle_(render_process),
report_ready_to_delete_(false),
event_handler_(event_handler),
- id_(id) {
+ id_(id),
+ video_capture_manager_(video_capture_manager) {
memset(&params_, 0, sizeof(params_));
}
@@ -35,21 +37,18 @@ void VideoCaptureController::StartCapture(
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
params_ = params;
- media_stream::VideoCaptureManager* manager =
- media_stream::MediaStreamManager::Get()->video_capture_manager();
// Order the manager to start the actual capture.
- manager->Start(params, this);
+ video_capture_manager_->Start(params, this);
}
void VideoCaptureController::StopCapture(Task* stopped_task) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
- media_stream::VideoCaptureManager* manager =
- media_stream::MediaStreamManager::Get()->video_capture_manager();
- manager->Stop(params_.session_id,
- NewRunnableMethod(this,
- &VideoCaptureController::OnDeviceStopped,
- stopped_task));
+ video_capture_manager_->Stop(
+ params_.session_id,
+ NewRunnableMethod(this,
+ &VideoCaptureController::OnDeviceStopped,
+ stopped_task));
}
void VideoCaptureController::ReturnBuffer(int buffer_id) {
@@ -158,8 +157,7 @@ void VideoCaptureController::OnIncomingCapturedFrame(const uint8* data,
void VideoCaptureController::OnError() {
event_handler_->OnError(id_);
- media_stream::MediaStreamManager::Get()->video_capture_manager()->
- Error(params_.session_id);
+ video_capture_manager_->Error(params_.session_id);
}
void VideoCaptureController::OnFrameInfo(

Powered by Google App Engine
This is Rietveld 408576698