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

Side by Side Diff: content/renderer/media/webrtc/media_stream_remote_video_source_unittest.cc

Issue 2466353003: Replace all use of cricket::VideoFrame and cricket::WebRtcVideoFrame with webrtc::VideoFrame. (Closed)
Patch Set: Drop unneeded include. Created 4 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/media/webrtc/media_stream_remote_video_source.h" 5 #include "content/renderer/media/webrtc/media_stream_remote_video_source.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/run_loop.h" 10 #include "base/run_loop.h"
11 #include "base/strings/utf_string_conversions.h" 11 #include "base/strings/utf_string_conversions.h"
12 #include "base/threading/thread_task_runner_handle.h" 12 #include "base/threading/thread_task_runner_handle.h"
13 #include "content/child/child_process.h" 13 #include "content/child/child_process.h"
14 #include "content/renderer/media/media_stream_video_track.h" 14 #include "content/renderer/media/media_stream_video_track.h"
15 #include "content/renderer/media/mock_media_stream_video_sink.h" 15 #include "content/renderer/media/mock_media_stream_video_sink.h"
16 #include "content/renderer/media/webrtc/mock_peer_connection_dependency_factory. h" 16 #include "content/renderer/media/webrtc/mock_peer_connection_dependency_factory. h"
17 #include "content/renderer/media/webrtc/track_observer.h" 17 #include "content/renderer/media/webrtc/track_observer.h"
18 #include "media/base/video_frame.h" 18 #include "media/base/video_frame.h"
19 #include "testing/gtest/include/gtest/gtest.h" 19 #include "testing/gtest/include/gtest/gtest.h"
20 #include "third_party/WebKit/public/web/WebHeap.h" 20 #include "third_party/WebKit/public/web/WebHeap.h"
21 #include "third_party/webrtc/media/engine/webrtcvideoframe.h"
22 21
23 namespace content { 22 namespace content {
24 23
25 ACTION_P(RunClosure, closure) { 24 ACTION_P(RunClosure, closure) {
26 closure.Run(); 25 closure.Run();
27 } 26 }
28 27
29 class MediaStreamRemoteVideoSourceUnderTest 28 class MediaStreamRemoteVideoSourceUnderTest
30 : public MediaStreamRemoteVideoSource { 29 : public MediaStreamRemoteVideoSource {
31 public: 30 public:
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
125 base::RunLoop run_loop; 124 base::RunLoop run_loop;
126 base::Closure quit_closure = run_loop.QuitClosure(); 125 base::Closure quit_closure = run_loop.QuitClosure();
127 EXPECT_CALL(sink, OnVideoFrame()).WillOnce( 126 EXPECT_CALL(sink, OnVideoFrame()).WillOnce(
128 RunClosure(quit_closure)); 127 RunClosure(quit_closure));
129 rtc::scoped_refptr<webrtc::I420Buffer> buffer( 128 rtc::scoped_refptr<webrtc::I420Buffer> buffer(
130 new rtc::RefCountedObject<webrtc::I420Buffer>(320, 240)); 129 new rtc::RefCountedObject<webrtc::I420Buffer>(320, 240));
131 130
132 buffer->SetToBlack(); 131 buffer->SetToBlack();
133 132
134 source()->SinkInterfaceForTest()->OnFrame( 133 source()->SinkInterfaceForTest()->OnFrame(
135 cricket::WebRtcVideoFrame(buffer, webrtc::kVideoRotation_0, 1000)); 134 webrtc::VideoFrame(buffer, webrtc::kVideoRotation_0, 1000));
136 run_loop.Run(); 135 run_loop.Run();
137 136
138 EXPECT_EQ(1, sink.number_of_frames()); 137 EXPECT_EQ(1, sink.number_of_frames());
139 track->RemoveSink(&sink); 138 track->RemoveSink(&sink);
140 } 139 }
141 140
142 TEST_F(MediaStreamRemoteVideoSourceTest, RemoteTrackStop) { 141 TEST_F(MediaStreamRemoteVideoSourceTest, RemoteTrackStop) {
143 std::unique_ptr<MediaStreamVideoTrack> track(CreateTrack()); 142 std::unique_ptr<MediaStreamVideoTrack> track(CreateTrack());
144 143
145 MockMediaStreamVideoSink sink; 144 MockMediaStreamVideoSink sink;
146 track->AddSink(&sink, sink.GetDeliverFrameCB(), false); 145 track->AddSink(&sink, sink.GetDeliverFrameCB(), false);
147 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateLive, sink.state()); 146 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateLive, sink.state());
148 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateLive, 147 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateLive,
149 webkit_source().getReadyState()); 148 webkit_source().getReadyState());
150 StopWebRtcTrack(); 149 StopWebRtcTrack();
151 base::RunLoop().RunUntilIdle(); 150 base::RunLoop().RunUntilIdle();
152 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateEnded, 151 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateEnded,
153 webkit_source().getReadyState()); 152 webkit_source().getReadyState());
154 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateEnded, sink.state()); 153 EXPECT_EQ(blink::WebMediaStreamSource::ReadyStateEnded, sink.state());
155 154
156 track->RemoveSink(&sink); 155 track->RemoveSink(&sink);
157 } 156 }
158 157
159 } // namespace content 158 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698