Resolves race between Channel::ProcessAndEncodeAudio() and Channel::StopSend()

Resolves race between Channel::ProcessAndEncodeAudio() and Channel::StopSend()


BUG=webrtc:7540
Review-Url: https://codereview.webrtc.org/2861583005
Cr-Commit-Position: refs/heads/master@{#17999}
Committed: https://chromium.googlesource.com/external/webrtc/+/4515fa0bed80d4a8e31836a73c14aade02bf68ba

[henrika_webrtc, 2017-05-03 12:53:59]

Description was changed from

==========
Resolves race between Channel::ProcessAndEncodeAudio() and Channel::StopSend()

Resolves race between Channel::ProcessAndEncodeAudio() and Channel::StopSend()


added logs

BUG=
==========

to

==========
Resolves race between Channel::ProcessAndEncodeAudio() and Channel::StopSend()


BUG=webrtc:7540
==========

[henrika_webrtc, 2017-05-03 12:54:00]

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

[henrika_webrtc, 2017-05-03 12:58:32]

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

[commit-bot: I haz the power, 2017-05-03 12:58:39]

Dry run: CQ is trying da patch.

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

[henrika_webrtc, 2017-05-03 12:58:55]

Why make it complicated ;-)

PTAL

[henrika_webrtc, 2017-05-03 13:05:16]

Wait. I might not be able to rely on the existing lock in ChannelState.
Let me check again.

[henrika_webrtc, 2017-05-03 13:58:54]

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

[commit-bot: I haz the power, 2017-05-03 13:59:01]

Dry run: CQ is trying da patch.

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

[henrika_webrtc, 2017-05-03 13:59:30]

OK. Please take a look now.

[the sun, 2017-05-03 14:55:25]

lgtm

[henrika_webrtc, 2017-05-03 15:10:32]

The CQ bit was unchecked by henrika@webrtc.org

[henrika_webrtc, 2017-05-03 15:10:36]

The CQ bit was checked by henrika@webrtc.org

[commit-bot: I haz the power, 2017-05-03 15:10:44]

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, 2017-05-03 15:30:15]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1493824236420900,
"parent_rev": "d41631aa274e926c4fba1600cbcfbc18020eb35a", "commit_rev":
"4515fa0bed80d4a8e31836a73c14aade02bf68ba"}

[commit-bot: I haz the power, 2017-05-03 15:30:19]

Description was changed from

==========
Resolves race between Channel::ProcessAndEncodeAudio() and Channel::StopSend()


BUG=webrtc:7540
==========

to

==========
Resolves race between Channel::ProcessAndEncodeAudio() and Channel::StopSend()


BUG=webrtc:7540

Review-Url: https://codereview.webrtc.org/2861583005
Cr-Commit-Position: refs/heads/master@{#17999}
Committed:
https://chromium.googlesource.com/external/webrtc/+/4515fa0bed80d4a8e31836a73...
==========

[commit-bot: I haz the power, 2017-05-03 15:30:20]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/external/webrtc/+/4515fa0bed80d4a8e31836a73...