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

Unified Diff: talk/session/media/channel.cc

Issue 1334793003: Remove VideoMediaChannel::SetRender(). (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: remove IsReadyToReceive from logging Created 5 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: talk/session/media/channel.cc
diff --git a/talk/session/media/channel.cc b/talk/session/media/channel.cc
index 1405c55bf78485210511b9add31fb5f62682d04c..5a6b7e198f16cb8380790e82d4eda8bc958e4c2a 100644
--- a/talk/session/media/channel.cc
+++ b/talk/session/media/channel.cc
@@ -1743,14 +1743,6 @@ bool VideoChannel::SetVideoSend(uint32 ssrc,
}
void VideoChannel::ChangeState() {
- // Render incoming data if we're the active call, and we have the local
- // content. We receive data on the default channel and multiplexed streams.
- bool recv = IsReadyToReceive();
- if (!media_channel()->SetRender(recv)) {
- LOG(LS_ERROR) << "Failed to SetRender on video channel";
- // TODO(gangji): Report error back to server.
- }
-
// Send outgoing data if we're the active call, we have the remote content,
// and we have had some form of connectivity.
bool send = IsReadyToSend();
@@ -1759,7 +1751,7 @@ void VideoChannel::ChangeState() {
// TODO(gangji): Report error back to server.
}
- LOG(LS_INFO) << "Changing video state, recv=" << recv << " send=" << send;
+ LOG(LS_INFO) << "Changing video state, send=" << send;
}
bool VideoChannel::GetStats(VideoMediaInfo* stats) {

Powered by Google App Engine
This is Rietveld 408576698