Revert of Fix getUserMedia DCHECK on multiple calls with stopped tracks (patchset...
commite1f32fd3e757795dd45618cce6a711c910140208
authorphoglund <phoglund@chromium.org>
Tue, 22 Sep 2015 09:27:10 +0000 (22 02:27 -0700)
committerCommit bot <commit-bot@chromium.org>
Tue, 22 Sep 2015 09:27:52 +0000 (22 09:27 +0000)
tree25876bc480b4e58b35ea2d74b80ee9296224e12c
parenta6994066613549825c4174fe9b72596e0c3aed7d
Revert of Fix getUserMedia DCHECK on multiple calls with stopped tracks (patchset #2 id:40001 of https://codereview.chromium.org/1349273002/ )

Reason for revert:
Speculative revert (could be breaking WebRTC AppRTC test): https://build.chromium.org/p/chromium.webrtc/builders/Mac%20Tester/builds/43297

Original issue's description:
> Fix getUserMedia DCHECK on multiple calls with stopped tracks
>
> There is a race occurring in situations similar to the following:
>
> getUserMedia(
>     {video: true},
>     function(stream) {
>         video_track = stream.getVideoTracks()[0];
>         getUserMedia(...);
>         video_track.stop(); },
>     function() {});
>
> This gets around the issue by postponing GenerateStream until after
> the track has finished stopping. See linked issues for more info.
>
> BUG=532554,525849
>
> Committed: https://crrev.com/aa8ccca3e1b5014807dc8bbc7174fd4ad982f773
> Cr-Commit-Position: refs/heads/master@{#349954}

TBR=perkj@chromium.org,mcasas@chromium.org,emircan@chromium.org,ajose@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=532554,525849

Review URL: https://codereview.chromium.org/1352093003

Cr-Commit-Position: refs/heads/master@{#350131}
content/renderer/media/user_media_client_impl.cc
content/renderer/media/user_media_client_impl.h
content/renderer/media/user_media_client_impl_unittest.cc