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

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

Issue 883293005: Cast: Basic cast_receiver API for chrome. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: missed include file changes Created 5 years, 10 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/renderer/media/media_stream_video_capture_source_unittest.cc
diff --git a/content/renderer/media/media_stream_video_capture_source_unittest.cc b/content/renderer/media/media_stream_video_capture_source_unittest.cc
index 6ebecadb37c58f3738e770f90e9ade2ea52a4910..242701b9228851df06ad0edd3f1154826be25f1d 100644
--- a/content/renderer/media/media_stream_video_capture_source_unittest.cc
+++ b/content/renderer/media/media_stream_video_capture_source_unittest.cc
@@ -24,12 +24,16 @@ class MockVideoCapturerDelegate : public VideoCapturerDelegate {
explicit MockVideoCapturerDelegate(const StreamDeviceInfo& device_info)
: VideoCapturerDelegate(device_info) {}
- MOCK_METHOD3(StartCapture,
- void(const media::VideoCaptureParams& params,
- const VideoCaptureDeliverFrameCB& new_frame_callback,
- const RunningCallback& running_callback));
+ MOCK_METHOD4(
+ StartCapture,
+ void(const media::VideoCaptureParams& params,
+ const VideoCaptureDeliverFrameCB& new_frame_callback,
+ scoped_refptr<base::SingleThreadTaskRunner>
+ frame_callback_task_runner,
+ const RunningCallback& running_callback));
MOCK_METHOD0(StopCapture, void());
+ protected:
virtual ~MockVideoCapturerDelegate() {}
};
@@ -48,14 +52,14 @@ class MediaStreamVideoCapturerSourceTest : public testing::Test {
}
void InitWithDeviceInfo(const StreamDeviceInfo& device_info) {
- scoped_ptr<MockVideoCapturerDelegate> delegate(
+ scoped_refptr<MockVideoCapturerDelegate> delegate(
new MockVideoCapturerDelegate(device_info));
delegate_ = delegate.get();
source_ = new MediaStreamVideoCapturerSource(
device_info,
base::Bind(&MediaStreamVideoCapturerSourceTest::OnSourceStopped,
base::Unretained(this)),
- delegate.Pass());
+ delegate);
webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"),
blink::WebMediaStreamSource::TypeVideo,
@@ -110,6 +114,7 @@ TEST_F(MediaStreamVideoCapturerSourceTest, TabCaptureAllowResolutionChange) {
testing::Field(&media::VideoCaptureParams::resolution_change_policy,
media::RESOLUTION_POLICY_DYNAMIC_WITHIN_LIMIT),
testing::_,
+ testing::_,
testing::_)).Times(1);
blink::WebMediaStreamTrack track = StartSource();
// When the track goes out of scope, the source will be stopped.
@@ -126,6 +131,7 @@ TEST_F(MediaStreamVideoCapturerSourceTest,
testing::Field(&media::VideoCaptureParams::resolution_change_policy,
media::RESOLUTION_POLICY_DYNAMIC_WITHIN_LIMIT),
testing::_,
+ testing::_,
testing::_)).Times(1);
blink::WebMediaStreamTrack track = StartSource();
// When the track goes out of scope, the source will be stopped.
@@ -135,14 +141,14 @@ TEST_F(MediaStreamVideoCapturerSourceTest,
TEST_F(MediaStreamVideoCapturerSourceTest, Ended) {
StreamDeviceInfo device_info;
device_info.device.type = MEDIA_DESKTOP_VIDEO_CAPTURE;
- scoped_ptr<VideoCapturerDelegate> delegate(
+ scoped_refptr<VideoCapturerDelegate> delegate(
new VideoCapturerDelegate(device_info));
VideoCapturerDelegate* delegate_ptr = delegate.get();
source_ = new MediaStreamVideoCapturerSource(
device_info,
base::Bind(&MediaStreamVideoCapturerSourceTest::OnSourceStopped,
base::Unretained(this)),
- delegate.Pass());
+ delegate);
webkit_source_.initialize(base::UTF8ToUTF16("dummy_source_id"),
blink::WebMediaStreamSource::TypeVideo,
base::UTF8ToUTF16("dummy_source_name"),
@@ -197,13 +203,14 @@ TEST_F(MediaStreamVideoCapturerSourceTest, CaptureTime) {
EXPECT_CALL(mock_delegate(), StartCapture(
testing::_,
testing::_,
+ testing::_,
testing::_))
.Times(1)
.WillOnce(testing::DoAll(testing::SaveArg<1>(&deliver_frame_cb),
- testing::SaveArg<2>(&running_cb)));
+ testing::SaveArg<3>(&running_cb)));
EXPECT_CALL(mock_delegate(), StopCapture());
blink::WebMediaStreamTrack track = StartSource();
- running_cb.Run(MEDIA_DEVICE_OK);
+ running_cb.Run(true);
base::RunLoop run_loop;
base::TimeTicks reference_capture_time =

Powered by Google App Engine
This is Rietveld 408576698