AudioRendererImpl: Fix data race.

AudioRendererImpl: Fix data race.

Previous CL introduced multithreaded access to |rendering_|.
https://codereview.chromium.org/2423903003/

BUG=657001, 648710
Committed: https://crrev.com/e2f429e24bdbcd8e8379b909879e2c729490e1c6
Cr-Commit-Position: refs/heads/master@{#426312}

[chcunningham, 2016-10-19 17:17:16]

chcunningham@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[chcunningham, 2016-10-19 17:17:16]

Hey Dale, worried I'm missing something here - why was rendering_ intentionally
accessed/set outside the lock (in most cases, only to have the method hold the
lock a few lines below)?

[DaleCurtis, 2016-10-19 17:21:29]

Sorry, don't remember, it's probably because it was only used on the main thread
previously.

[chcunningham, 2016-10-19 18:19:00]

chcunningham@chromium.org changed reviewers:
+ sandersd@chromium.org
- dalecurtis@chromium.org

[chcunningham, 2016-10-19 18:19:00]

On 2016/10/19 17:21:29, DaleCurtis wrote:
> Sorry, don't remember, it's probably because it was only used on the main
thread
> previously.

Ok, thanks.

Dale -> CC
Dan (who did previous review) -> reviewer

[sandersd (OOO until July 31), 2016-10-19 18:20:25]

lgtm

[chcunningham, 2016-10-19 18:30:59]

Thanks Dan

[chcunningham, 2016-10-19 18:31:04]

The CQ bit was checked by chcunningham@chromium.org

[commit-bot: I haz the power, 2016-10-19 18:32:02]

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, 2016-10-19 19:50:06]

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

[commit-bot: I haz the power, 2016-10-19 19:50:07]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[chcunningham, 2016-10-19 20:44:38]

The CQ bit was checked by chcunningham@chromium.org

[commit-bot: I haz the power, 2016-10-19 20:45:08]

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, 2016-10-19 22:41:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-21 13:12:25]

Description was changed from

==========
AudioRendererImpl: Fix data race.

Previous CL introduced multithreaded access to |rendering_|.
https://codereview.chromium.org/2423903003/

BUG=657001,648710
==========

to

==========
AudioRendererImpl: Fix data race.

Previous CL introduced multithreaded access to |rendering_|.
https://codereview.chromium.org/2423903003/

BUG=657001,648710

Committed: https://crrev.com/e2f429e24bdbcd8e8379b909879e2c729490e1c6
Cr-Commit-Position: refs/heads/master@{#426312}
==========

[commit-bot: I haz the power, 2016-10-21 13:12:26]

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