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

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

Issue 2393923002: Add unit tests for webrtc_media_stream_adapter.cc (Closed)
Patch Set: Renamed test function Created 4 years, 2 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/media_stream_video_webrtc_sink.cc
diff --git a/content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc b/content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc
index cafc3facc17b20a07f2a858b617bcd1dadbc1e37..99db0f2c3ee77976d518bb820a65e109f563d0d0 100644
--- a/content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc
+++ b/content/renderer/media/webrtc/media_stream_video_webrtc_sink.cc
@@ -206,6 +206,7 @@ MediaStreamVideoWebRtcSink::MediaStreamVideoWebRtcSink(
const blink::WebMediaStreamTrack& track,
PeerConnectionDependencyFactory* factory)
: weak_factory_(this) {
+ DCHECK(MediaStreamVideoTrack::GetVideoTrack(track));
const blink::WebMediaConstraints& constraints =
MediaStreamVideoTrack::GetVideoTrack(track)->constraints();
@@ -312,4 +313,9 @@ void MediaStreamVideoWebRtcSink::RequestRefreshFrame() {
content::RequestRefreshFrameFromVideoTrack(connected_track());
}
+rtc::Optional<bool> MediaStreamVideoWebRtcSink::SourceNeedsDenoisingForTesting()
+ const {
+ return video_source_->needs_denoising();
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698