Relaxed unnecessarily stringent thread checking in WebRtcAudioSendStream::OnData().

Relaxed unnecessarily stringent thread checking in WebRtcAudioSendStream::OnData().

BUG=webrtc:6345
Committed: https://crrev.com/6fa69c91d6192e242ed2bc70836a4f73132a958b
Cr-Commit-Position: refs/heads/master@{#14214}

[the sun, 2016-09-14 07:19:15]

solenberg@webrtc.org changed reviewers:
+ henrikg@webrtc.org

[Henrik Grunell WebRTC, 2016-09-14 07:27:04]

lgtm

https://codereview.webrtc.org/2332213006/diff/1/webrtc/media/engine/webrtcvoi...
File webrtc/media/engine/webrtcvoiceengine.cc (left):

https://codereview.webrtc.org/2332213006/diff/1/webrtc/media/engine/webrtcvoi...
webrtc/media/engine/webrtcvoiceengine.cc:1273:
RTC_DCHECK(!worker_thread_checker_.CalledOnValidThread());
Just for the record: this is not the check that fails in the bug. It's the below
line. Still valid change though.

[the sun, 2016-09-14 07:54:28]

On 2016/09/14 07:27:04, Henrik Grunell (webrtc) wrote:
> lgtm
> 
>
https://codereview.webrtc.org/2332213006/diff/1/webrtc/media/engine/webrtcvoi...
> File webrtc/media/engine/webrtcvoiceengine.cc (left):
> 
>
https://codereview.webrtc.org/2332213006/diff/1/webrtc/media/engine/webrtcvoi...
> webrtc/media/engine/webrtcvoiceengine.cc:1273:
> RTC_DCHECK(!worker_thread_checker_.CalledOnValidThread());
> Just for the record: this is not the check that fails in the bug. It's the
below
> line. Still valid change though.

Yes, I realized, but this looks like it will bite us sooner than later so I'll
land the change instead of adding a comment about it.

[the sun, 2016-09-14 07:54:32]

The CQ bit was checked by solenberg@webrtc.org

[commit-bot: I haz the power, 2016-09-14 07:54:36]

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-14 09:55:06]

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

[commit-bot: I haz the power, 2016-09-14 09:55:07]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[the sun, 2016-09-14 12:53:29]

The CQ bit was checked by solenberg@webrtc.org

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

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-14 13:01:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-14 13:01:42]

Description was changed from

==========
Relaxed unnecessarily stringent thread checking in
WebRtcAudioSendStream::OnData().

BUG=webrtc:6345
==========

to

==========
Relaxed unnecessarily stringent thread checking in
WebRtcAudioSendStream::OnData().

BUG=webrtc:6345

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

[commit-bot: I haz the power, 2016-09-14 13:01:43]

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