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

Unified Diff: content/renderer/media/mock_media_stream_dependency_factory.cc

Issue 201583003: Implement a source for remote video tracks. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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/mock_media_stream_dependency_factory.cc
diff --git a/content/renderer/media/mock_media_stream_dependency_factory.cc b/content/renderer/media/mock_media_stream_dependency_factory.cc
index 92192b94cd40fc1bb50871566bacd3869068302d..65c2c328c0a0975bfe840b9e451bbe6bdc82c764 100644
--- a/content/renderer/media/mock_media_stream_dependency_factory.cc
+++ b/content/renderer/media/mock_media_stream_dependency_factory.cc
@@ -292,17 +292,20 @@ MockLocalVideoTrack::MockLocalVideoTrack(std::string id,
id_(id),
state_(MediaStreamTrackInterface::kLive),
source_(source),
- observer_(NULL) {
+ observer_(NULL),
+ renderer_(NULL) {
}
MockLocalVideoTrack::~MockLocalVideoTrack() {}
void MockLocalVideoTrack::AddRenderer(VideoRendererInterface* renderer) {
- NOTIMPLEMENTED();
+ DCHECK(!renderer_);
+ renderer_ = renderer;
}
void MockLocalVideoTrack::RemoveRenderer(VideoRendererInterface* renderer) {
Jói 2014/03/17 22:00:40 The interface here is a little weird unless you pl
perkj_chrome 2014/03/19 16:34:48 This is an overriden method. The real implementati
- NOTIMPLEMENTED();
+ DCHECK_EQ(renderer_, renderer);
+ renderer_ = NULL;
}
std::string MockLocalVideoTrack::kind() const {

Powered by Google App Engine
This is Rietveld 408576698