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

Unified Diff: content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc

Issue 2356663002: Implement and use VideoTrackSource directly. (Closed)
Patch Set: add test TODO Created 4 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/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc
diff --git a/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc b/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc
index bbffbc6eb553eefbc63d6b23dd8f4afbefd88a37..80ff16c6dffed88a475d35fabcdffc06296390af 100644
--- a/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc
+++ b/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc
@@ -119,39 +119,6 @@ void MockMediaStream::NotifyObservers() {
MockMediaStream::~MockMediaStream() {}
-class MockRtcVideoCapturer : public WebRtcVideoCapturerAdapter {
- public:
- explicit MockRtcVideoCapturer(bool is_screencast)
- : WebRtcVideoCapturerAdapter(is_screencast),
- number_of_capturered_frames_(0),
- width_(0),
- height_(0) {
- }
-
- void OnFrameCaptured(const scoped_refptr<media::VideoFrame>& frame) override {
- ++number_of_capturered_frames_;
- width_ = frame->visible_rect().width();
- height_ = frame->visible_rect().height();
- }
-
- int GetLastFrameWidth() const {
- return width_;
- }
-
- int GetLastFrameHeight() const {
- return height_;
- }
-
- int GetFrameNum() const {
- return number_of_capturered_frames_;
- }
-
- private:
- int number_of_capturered_frames_;
- int width_;
- int height_;
-};
-
scoped_refptr<MockWebRtcAudioTrack> MockWebRtcAudioTrack::Create(
const std::string& id) {
return new rtc::RefCountedObject<MockWebRtcAudioTrack>(id);
@@ -367,21 +334,11 @@ MockPeerConnectionDependencyFactory::CreatePeerConnection(
return new rtc::RefCountedObject<MockPeerConnectionImpl>(this, observer);
}
-WebRtcVideoCapturerAdapter*
-MockPeerConnectionDependencyFactory::CreateVideoCapturer(
- bool is_screen_capture) {
- return new MockRtcVideoCapturer(is_screen_capture);
-}
-
scoped_refptr<webrtc::VideoTrackSourceInterface>
-MockPeerConnectionDependencyFactory::CreateVideoSource(
- cricket::VideoCapturer* capturer) {
- // Video source normally take ownership of |capturer|.
- delete capturer;
- NOTIMPLEMENTED();
+MockPeerConnectionDependencyFactory::CreateVideoTrackSourceProxy(
+ webrtc::VideoTrackSourceInterface* source) {
return nullptr;
}
-
scoped_refptr<webrtc::MediaStreamInterface>
MockPeerConnectionDependencyFactory::CreateLocalMediaStream(
const std::string& label) {
@@ -398,13 +355,6 @@ MockPeerConnectionDependencyFactory::CreateLocalVideoTrack(
return track;
}
-scoped_refptr<webrtc::VideoTrackInterface>
-MockPeerConnectionDependencyFactory::CreateLocalVideoTrack(
- const std::string& id,
- cricket::VideoCapturer* capturer) {
- return new rtc::RefCountedObject<MockWebRtcVideoTrack>(id, nullptr);
-}
-
SessionDescriptionInterface*
MockPeerConnectionDependencyFactory::CreateSessionDescription(
const std::string& type,

Powered by Google App Engine
This is Rietveld 408576698