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

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

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.h
diff --git a/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.h b/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.h
index a4ef6d162c8498a244064fc00e8dd0fb881a7deb..35b8b7af5e3b75d0f848a9fa83e87997ca3e6d03 100644
--- a/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.h
+++ b/content/renderer/media/webrtc/mock_peer_connection_dependency_factory.h
@@ -35,8 +35,7 @@ class MockVideoRenderer : public cricket::VideoRenderer {
class MockAudioSource : public webrtc::AudioSourceInterface {
public:
- explicit MockAudioSource(
- const webrtc::MediaConstraintsInterface* constraints, bool remote);
+ explicit MockAudioSource(const cricket::AudioOptions& options, bool remote);
void RegisterObserver(webrtc::ObserverInterface* observer) override;
void UnregisterObserver(webrtc::ObserverInterface* observer) override;
@@ -48,15 +47,6 @@ class MockAudioSource : public webrtc::AudioSourceInterface {
// Changes the state of the source to ended and notifies the observer.
void SetEnded();
- const webrtc::MediaConstraintsInterface::Constraints& optional_constraints() {
- return optional_constraints_;
- }
-
- const webrtc::MediaConstraintsInterface::Constraints&
- mandatory_constraints() {
- return mandatory_constraints_;
- }
-
protected:
~MockAudioSource() override;
@@ -64,8 +54,6 @@ class MockAudioSource : public webrtc::AudioSourceInterface {
bool remote_;
ObserverSet observers_;
MediaSourceInterface::SourceState state_;
- webrtc::MediaConstraintsInterface::Constraints optional_constraints_;
- webrtc::MediaConstraintsInterface::Constraints mandatory_constraints_;
};
class MockWebRtcVideoTrack : public webrtc::VideoTrackInterface {
@@ -137,16 +125,14 @@ class MockPeerConnectionDependencyFactory
scoped_refptr<webrtc::PeerConnectionInterface> CreatePeerConnection(
const webrtc::PeerConnectionInterface::RTCConfiguration& config,
- const webrtc::MediaConstraintsInterface* constraints,
blink::WebFrame* frame,
webrtc::PeerConnectionObserver* observer) override;
scoped_refptr<webrtc::AudioSourceInterface> CreateLocalAudioSource(
- const webrtc::MediaConstraintsInterface* constraints) override;
+ const cricket::AudioOptions& options) override;
WebRtcVideoCapturerAdapter* CreateVideoCapturer(
bool is_screen_capture) override;
scoped_refptr<webrtc::VideoTrackSourceInterface> CreateVideoSource(
- cricket::VideoCapturer* capturer,
- const blink::WebMediaConstraints& constraints) override;
+ cricket::VideoCapturer* capturer) override;
void CreateWebAudioSource(blink::WebMediaStreamSource* source) override;
scoped_refptr<webrtc::MediaStreamInterface> CreateLocalMediaStream(
const std::string& label) override;

Powered by Google App Engine
This is Rietveld 408576698