Lock out GCs while iterating over MediaStreamSource audio consumers.

Lock out GCs while iterating over MediaStreamSource audio consumers.

Attempt to diagnose a crash condition by locking out main thread GCs
while the audio thread propagates consumeAudio() updates.

R=haraken
BUG=682945
Review-Url: https://codereview.chromium.org/2761463002
Cr-Commit-Position: refs/heads/master@{#457753}
Committed: https://chromium.googlesource.com/chromium/src/+/e78674d2467706fa8634f8542251591d1d358b57

[sof, 2017-03-17 10:55:50]

The CQ bit was checked by sigbjornf@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2017-03-17 10:56:02]

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

[sof, 2017-03-17 11:09:15]

sigbjornf@opera.com changed reviewers:
+ haraken@chromium.org

[sof, 2017-03-17 11:09:15]

please take a look.

i'd like to try; we could make this more focused and just outlaw heap
compaction, i suppose.

[haraken, 2017-03-17 11:40:44]

LGTM

[commit-bot: I haz the power, 2017-03-17 12:46:27]

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

[commit-bot: I haz the power, 2017-03-17 12:46:28]

Dry run: This issue passed the CQ dry run.

[sof, 2017-03-17 13:44:07]

Description was changed from

==========
Lock out GCs while iterating over MediaStreamSource audio consumers.

Attempt to diagnose a crash condition by locking out main thread GCs
while the audio thread propagates consumeAudio() updates.

R=
BUG=682945
==========

to

==========
Lock out GCs while iterating over MediaStreamSource audio consumers.

Attempt to diagnose a crash condition by locking out main thread GCs
while the audio thread propagates consumeAudio() updates.

R=haraken
BUG=682945
==========

[sof, 2017-03-17 13:44:12]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2017-03-17 13:44:31]

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

[commit-bot: I haz the power, 2017-03-17 13:48:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489758252503660, "parent_rev":
"6955cae058e32565a628bb9c0c37f01aad23cdf3", "commit_rev":
"e78674d2467706fa8634f8542251591d1d358b57"}

[commit-bot: I haz the power, 2017-03-17 13:48:47]

Description was changed from

==========
Lock out GCs while iterating over MediaStreamSource audio consumers.

Attempt to diagnose a crash condition by locking out main thread GCs
while the audio thread propagates consumeAudio() updates.

R=haraken
BUG=682945
==========

to

==========
Lock out GCs while iterating over MediaStreamSource audio consumers.

Attempt to diagnose a crash condition by locking out main thread GCs
while the audio thread propagates consumeAudio() updates.

R=haraken
BUG=682945

Review-Url: https://codereview.chromium.org/2761463002
Cr-Commit-Position: refs/heads/master@{#457753}
Committed:
https://chromium.googlesource.com/chromium/src/+/e78674d2467706fa8634f8542251...
==========

[commit-bot: I haz the power, 2017-03-17 13:48:48]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e78674d2467706fa8634f8542251...

[sof, 2017-03-22 09:51:09]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2768683004/ by sigbjornf@opera.com.

The reason for reverting is: Revert this check, no longer needed (cf.
https://bugs.chromium.org/p/chromium/issues/detail?id=682945#c48 ).