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

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

Issue 10537057: refactor VideoCaptureHost and VideoCaptureController. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 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/video_capture_controller_unittest.cc
===================================================================
--- content/browser/renderer_host/media/video_capture_controller_unittest.cc (revision 141042)
+++ content/browser/renderer_host/media/video_capture_controller_unittest.cc (working copy)
@@ -31,11 +31,11 @@
enum { kDeviceId = 1 };
-ACTION_P5(StopCapture, controller, controller_id, controller_handler, flag,
+ACTION_P4(StopCapture, controller, controller_id, controller_handler,
message_loop) {
message_loop->PostTask(FROM_HERE,
base::Bind(&VideoCaptureController::StopCapture,
- controller, controller_id, controller_handler, flag));
+ controller, controller_id, controller_handler));
message_loop->PostTask(FROM_HERE, MessageLoop::QuitClosure());
}
@@ -90,7 +90,6 @@
EXPECT_EQ(id, controller_id_);
DoPaused(id);
}
- virtual void OnReadyToDelete(const VideoCaptureControllerID& id) OVERRIDE {}
scoped_refptr<VideoCaptureController> controller_;
MessageLoop* message_loop_;
@@ -206,7 +205,7 @@
.Times(AtLeast(1))
.WillOnce(StopCapture(controller_.get(),
controller_handler_->controller_id_,
- controller_handler_.get(), true,
+ controller_handler_.get(),
message_loop_.get()));
EXPECT_CALL(*vcm_,
StopCapture(vcm_->video_session_id_))
@@ -268,5 +267,5 @@
StopCapture(vcm_->video_session_id_))
.Times(1);
controller_->StopCapture(controller_handler_->controller_id_,
- controller_handler_.get(), true);
+ controller_handler_.get());
}

Powered by Google App Engine
This is Rietveld 408576698