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

Unified Diff: webrtc/pc/mediacontroller.h

Issue 2675173003: Adding "adapter" ORTC objects on top of ChannelManager/BaseChannel/etc. (Closed)
Patch Set: Add memcheck suppression for end-to-end tests. Created 3 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
« no previous file with comments | « webrtc/pc/channelmanager.cc ('k') | webrtc/pc/mediasession.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/pc/mediacontroller.h
diff --git a/webrtc/pc/mediacontroller.h b/webrtc/pc/mediacontroller.h
index 85617aff0acba778dc5efa2ef0dc395a1986f290..f5a9034918b1df58ab55115a11d91d10168da5b1 100644
--- a/webrtc/pc/mediacontroller.h
+++ b/webrtc/pc/mediacontroller.h
@@ -23,10 +23,12 @@ class Call;
class VoiceEngine;
class RtcEventLog;
-// The MediaController currently owns shared state between media channels, but
-// in the future will create and own RtpSenders and RtpReceivers.
+// The MediaController currently owns shared state between media channels.
+// Abstract interface is defined here such that it can be faked/mocked for
+// tests, but no other real reason.
class MediaControllerInterface {
public:
+ // Will never return nullptr.
static MediaControllerInterface* Create(
const cricket::MediaConfig& config,
rtc::Thread* worker_thread,
« no previous file with comments | « webrtc/pc/channelmanager.cc ('k') | webrtc/pc/mediasession.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698