Commit 6c7db377 authored by sakal's avatar sakal Committed by Commit bot

Remove source->Stop() call from chromoting WebRTC video stream.

Removing this call should be safe because the call just does following
things:
1. Sets started_ to false in VideoCapturerTrackSource. This doesn't
matter since it will be destroyed soon.
2. Calls WebRtcDummyVideoCapturer::Stop. This just sets the state of the
dummy video captuer to CS_STOPPED. This doesn't matter since it will be
destroyed soon.

Motivation behind this change is that we want to remove Restart and Stop
from VideoTrackSourceInterface. This code is the last usage of these
methods.

Review-Url: https://codereview.chromium.org/2308543002
Cr-Commit-Position: refs/heads/master@{#416290}
parent dd8e0f19
......@@ -69,7 +69,6 @@ WebrtcVideoStream::WebrtcVideoStream()
WebrtcVideoStream::~WebrtcVideoStream() {
if (stream_) {
for (const auto& track : stream_->GetVideoTracks()) {
track->GetSource()->Stop();
stream_->RemoveTrack(track.get());
}
peer_connection_->RemoveStream(stream_.get());
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment