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

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

Issue 1729683002: Remove old-style constraints from Chrome internals (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Complete the interface change Created 4 years, 9 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 a268fc52ccd027c8bb8a05029f94c5d731ecd6fc..f1f6095c60ea865a58ba498c31bb5e3e927d7541 100644
--- a/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc
+++ b/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.cc
@@ -167,12 +167,9 @@ bool MockVideoRenderer::RenderFrame(const cricket::VideoFrame* frame) {
return true;
}
-MockAudioSource::MockAudioSource(
- const webrtc::MediaConstraintsInterface* constraints, bool remote)
- : remote_(remote), state_(MediaSourceInterface::kLive),
- optional_constraints_(constraints->GetOptional()),
- mandatory_constraints_(constraints->GetMandatory()) {
-}
+MockAudioSource::MockAudioSource(const cricket::AudioOptions& options,
+ bool remote)
+ : remote_(remote), state_(MediaSourceInterface::kLive) {}
MockAudioSource::~MockAudioSource() {}
@@ -340,7 +337,6 @@ MockPeerConnectionDependencyFactory::~MockPeerConnectionDependencyFactory() {}
scoped_refptr<webrtc::PeerConnectionInterface>
MockPeerConnectionDependencyFactory::CreatePeerConnection(
const webrtc::PeerConnectionInterface::RTCConfiguration& config,
- const webrtc::MediaConstraintsInterface* constraints,
blink::WebFrame* frame,
webrtc::PeerConnectionObserver* observer) {
return new rtc::RefCountedObject<MockPeerConnectionImpl>(this, observer);
@@ -348,9 +344,9 @@ MockPeerConnectionDependencyFactory::CreatePeerConnection(
scoped_refptr<webrtc::AudioSourceInterface>
MockPeerConnectionDependencyFactory::CreateLocalAudioSource(
- const webrtc::MediaConstraintsInterface* constraints) {
+ const cricket::AudioOptions& options) {
last_audio_source_ =
- new rtc::RefCountedObject<MockAudioSource>(constraints, false);
+ new rtc::RefCountedObject<MockAudioSource>(options, false);
return last_audio_source_;
}
@@ -362,8 +358,7 @@ MockPeerConnectionDependencyFactory::CreateVideoCapturer(
scoped_refptr<webrtc::VideoTrackSourceInterface>
MockPeerConnectionDependencyFactory::CreateVideoSource(
- cricket::VideoCapturer* capturer,
- const blink::WebMediaConstraints& constraints) {
+ cricket::VideoCapturer* capturer) {
// Video source normally take ownership of |capturer|.
delete capturer;
NOTIMPLEMENTED();

Powered by Google App Engine
This is Rietveld 408576698