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

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.

Committed: https://crrev.com/6c7db377353fed02f6ac2052e86ff8ad89729245
Cr-Commit-Position: refs/heads/master@{#416290}

[sakal-chromium, 2016-09-02 11:24:58]

Description was changed from

==========
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 VideoTrackSource interface. This code is the last usage of these
methods.
==========

to

==========
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 VideoTrackSource interface. This code is the last usage of these
methods.
==========

[sakal-chromium, 2016-09-02 11:24:58]

sakal@chromium.org changed reviewers:
+ magjed@webrtc.org, sergeyu@chromium.org

[sakal-chromium, 2016-09-02 11:24:58]

sakal@chromium.org changed required reviewers:
+ sergeyu@chromium.org

[sakal-chromium, 2016-09-02 11:25:19]

sakal@chromium.org changed reviewers:
+ magjed@chromium.org
- magjed@webrtc.org

[sakal-chromium, 2016-09-02 11:26:05]

The CQ bit was checked by sakal@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-02 11:26:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 11:26:28]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 12:17:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-02 12:17:54]

Dry run: This issue passed the CQ dry run.

[sakal-chromium, 2016-09-02 12:37:28]

PTAL

[magjed_chromium, 2016-09-02 12:41:30]

lgtm

[sakal-chromium, 2016-09-02 12:42:23]

Description was changed from

==========
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 VideoTrackSource interface. This code is the last usage of these
methods.
==========

to

==========
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.
==========

[Sergey Ulanov, 2016-09-02 17:21:40]

lgtm

[Sergey Ulanov, 2016-09-02 17:22:04]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2016-09-02 17:22:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 17:26:50]

Description was changed from

==========
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.
==========

to

==========
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.
==========

[commit-bot: I haz the power, 2016-09-02 17:26:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-02 17:29:56]

Description was changed from

==========
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.
==========

to

==========
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.

Committed: https://crrev.com/6c7db377353fed02f6ac2052e86ff8ad89729245
Cr-Commit-Position: refs/heads/master@{#416290}
==========

[commit-bot: I haz the power, 2016-09-02 17:29:57]

Patchset 1 (id:??) landed as
https://crrev.com/6c7db377353fed02f6ac2052e86ff8ad89729245
Cr-Commit-Position: refs/heads/master@{#416290}