Remove stop method from VideoTrackSourceInterface.

Remove stop method from VideoTrackSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/

Committed: https://crrev.com/859e86107d71e7702651e2a2f738b7ff2fd58be0
Cr-Commit-Position: refs/heads/master@{#14085}

BUG=webrtc:6318

[sakal, 2016-09-05 10:38:18]

Description was changed from

==========
Renimove stop method from VideoTrakSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/
==========

to

==========
Remove stop method from VideoTrakSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/
==========

[sakal, 2016-09-05 10:39:32]

sakal@webrtc.org changed reviewers:
+ magjed@webrtc.org, tommi@webrtc.org

[sakal, 2016-09-05 10:40:17]

PTAL

[tommi, 2016-09-05 13:14:11]

lgtm

[sakal, 2016-09-05 13:16:27]

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

[commit-bot: I haz the power, 2016-09-05 13:16:38]

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

[magjed_webrtc, 2016-09-05 14:25:42]

lgtm

nit: The title is misspelled, add a 'c' in VideoTrakSourceInterface.

https://codereview.webrtc.org/2312603003/diff/1/webrtc/api/videocapturertrack...
File webrtc/api/videocapturertracksource.h (right):

https://codereview.webrtc.org/2312603003/diff/1/webrtc/api/videocapturertrack...
webrtc/api/videocapturertracksource.h:68: void Stop();
This function is only called in one place. You can consider inlining it in
~VideoCapturerTrackSource() directly.

[sakal, 2016-09-05 14:27:13]

Description was changed from

==========
Remove stop method from VideoTrakSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/
==========

to

==========
Remove stop method from VideoTrackSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/
==========

[commit-bot: I haz the power, 2016-09-05 15:50:20]

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

[commit-bot: I haz the power, 2016-09-05 15:50:21]

Dry run: This issue passed the CQ dry run.

[sakal, 2016-09-06 09:12:43]

The CQ bit was checked by sakal@webrtc.org

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

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

[commit-bot: I haz the power, 2016-09-06 09:25:09]

Description was changed from

==========
Remove stop method from VideoTrackSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/
==========

to

==========
Remove stop method from VideoTrackSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/
==========

[commit-bot: I haz the power, 2016-09-06 09:25:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-06 09:25:17]

Description was changed from

==========
Remove stop method from VideoTrackSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/
==========

to

==========
Remove stop method from VideoTrackSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/

Committed: https://crrev.com/859e86107d71e7702651e2a2f738b7ff2fd58be0
Cr-Commit-Position: refs/heads/master@{#14085}
==========

[commit-bot: I haz the power, 2016-09-06 09:25:18]

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

[tommi, 2016-09-06 13:15:38]

since this cl does modify the public API, I think we should have a bug for
tracking it

[sakal, 2016-09-06 13:22:12]

Description was changed from

==========
Remove stop method from VideoTrackSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/

Committed: https://crrev.com/859e86107d71e7702651e2a2f738b7ff2fd58be0
Cr-Commit-Position: refs/heads/master@{#14085}
==========

to

==========
Remove stop method from VideoTrackSourceInterface.

This method is no longer needed.

Follow-up on CL: https://codereview.webrtc.org/2306013002/

Committed: https://crrev.com/859e86107d71e7702651e2a2f738b7ff2fd58be0
Cr-Commit-Position: refs/heads/master@{#14085}

BUG=webrtc:6318
==========

[sakal, 2016-09-06 13:22:28]

On 2016/09/06 13:15:38, tommi (webrtc) wrote:
> since this cl does modify the public API, I think we should have a bug for
> tracking it

Done.

https://bugs.chromium.org/p/webrtc/issues/detail?id=6318