MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not 'live'

MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not 'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
TEST= See bug #1:

Steps to reproduce:
1- Go to https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html
 (note: needs either --enable-blink-features=GetUserMedia flag or
 chrome://flags -> Experimental Web Platform features enabled)
2- Click on GetUserMedia
3- Click on StopLocalAudioTracks to disable all audio tracks.
4- Click on CreateRecorder
5- Click on Start

Committed: https://crrev.com/2181b9dfb8af9622a2a87afda9bbc10717721f0b
Cr-Commit-Position: refs/heads/master@{#382697}

[mcasas, 2016-03-22 21:31:51]

mcasas@chromium.org changed reviewers:
+ niklase@chromium.org

[mcasas, 2016-03-22 21:31:52]

niklase@ PTAL

[mcasas, 2016-03-22 21:33:49]

Patchset #1 (id:1) has been deleted

[Niklas Enbom, 2016-03-22 21:36:30]

Tested? TEST=Load
https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html,
click GetUserMedia, StoplocalAudioTracks, CreateRecorder,Start


On 2016/03/22 21:31:52, mcasas wrote:
> niklase@ PTAL

[mcasas, 2016-03-22 21:38:45]

Description was changed from

==========
MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not
'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
==========

to

==========
MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not
'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
TEST= See bug #1:

Steps to reproduce:
1- Go to
https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html
2- Click on GetUserMedia
3- Click on StopLocalAudioTracks to disable all audio tracks.
4- Click on CreateRecorder
5- Click on Start
==========

[mcasas, 2016-03-22 21:48:34]

Description was changed from

==========
MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not
'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
TEST= See bug #1:

Steps to reproduce:
1- Go to
https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html
2- Click on GetUserMedia
3- Click on StopLocalAudioTracks to disable all audio tracks.
4- Click on CreateRecorder
5- Click on Start
==========

to

==========
MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not
'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
TEST= See bug #1:

Steps to reproduce:
1- Go to
https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html
 (note: needs either --enable-blink-features=GetUserMedia flag or
 chrome://flags -> Experimental Web Platform features enabled)
2- Click on GetUserMedia
3- Click on StopLocalAudioTracks to disable all audio tracks.
4- Click on CreateRecorder
5- Click on Start
==========

[mcasas, 2016-03-22 21:48:58]

niklase@: tested as described and working.
PTAL

[Niklas Enbom, 2016-03-22 21:54:53]

lgtm
On 2016/03/22 21:48:58, mcasas wrote:
> niklase@: tested as described and working.
> PTAL

[Niklas Enbom, 2016-03-22 21:54:58]

The CQ bit was checked by niklase@chromium.org

[commit-bot: I haz the power, 2016-03-22 21:55:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1824933005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1824933005/20001

[commit-bot: I haz the power, 2016-03-22 22:00:52]

Description was changed from

==========
MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not
'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
TEST= See bug #1:

Steps to reproduce:
1- Go to
https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html
 (note: needs either --enable-blink-features=GetUserMedia flag or
 chrome://flags -> Experimental Web Platform features enabled)
2- Click on GetUserMedia
3- Click on StopLocalAudioTracks to disable all audio tracks.
4- Click on CreateRecorder
5- Click on Start
==========

to

==========
MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not
'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
TEST= See bug #1:

Steps to reproduce:
1- Go to
https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html
 (note: needs either --enable-blink-features=GetUserMedia flag or
 chrome://flags -> Experimental Web Platform features enabled)
2- Click on GetUserMedia
3- Click on StopLocalAudioTracks to disable all audio tracks.
4- Click on CreateRecorder
5- Click on Start
==========

[commit-bot: I haz the power, 2016-03-22 22:00:53]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-03-22 22:03:43]

Description was changed from

==========
MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not
'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
TEST= See bug #1:

Steps to reproduce:
1- Go to
https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html
 (note: needs either --enable-blink-features=GetUserMedia flag or
 chrome://flags -> Experimental Web Platform features enabled)
2- Click on GetUserMedia
3- Click on StopLocalAudioTracks to disable all audio tracks.
4- Click on CreateRecorder
5- Click on Start
==========

to

==========
MediaStreamTrack: don't wait to record MediaStreamTrack's whose state is not
'live'

A MSTrack on which stop() has been called has state 'ended'.
With this CL MR avoids waiting for those Tracks.

BUG=596178
TEST= See bug #1:

Steps to reproduce:
1- Go to
https://cdn.rawgit.com/cricdecyan/mediarecorder/master/manualtest/index.html
 (note: needs either --enable-blink-features=GetUserMedia flag or
 chrome://flags -> Experimental Web Platform features enabled)
2- Click on GetUserMedia
3- Click on StopLocalAudioTracks to disable all audio tracks.
4- Click on CreateRecorder
5- Click on Start

Committed: https://crrev.com/2181b9dfb8af9622a2a87afda9bbc10717721f0b
Cr-Commit-Position: refs/heads/master@{#382697}
==========

[commit-bot: I haz the power, 2016-03-22 22:03:44]

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