Revert of Add lock to fix race in AudioRendererMixerInput.

Revert of Add lock to fix race in AudioRendererMixerInput. (patchset #5 id:80001 of https://codereview.chromium.org/1748183006/ )

Reason for revert:
This seems to cause failures in LayoutTests/http/tests/security/media. Not sure why this wasn't caught by CQ...

Tracking CQ weirdness here:
https://bugs.chromium.org/p/chromium/issues/detail?id=592079

Original issue's description:
> Add lock to fix race in AudioRendererMixerInput.
>
> Clusterfuzz identified a race between the media thread calling SetVolume
> and the audio device thread calling ProvideInput. Add a lock to
> synchronize access to |volume_| between threads.
>
> Also adds some thread_checkers to just firm up the contract that the
> majority of these methods are called only by the media thread.
>
> BUG=588992
>
> Committed: https://crrev.com/57c6958a99f2c62a4408d2a9d262a22129309106
> Cr-Commit-Position: refs/heads/master@{#379349}

TBR=dalecurtis@chromium.org,olka@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=588992
Committed: https://crrev.com/6b206d7b4963be6e8e7ff269dcf0015eb7d8bffa
Cr-Commit-Position: refs/heads/master@{#379388}

[chcunningham, 2016-03-04 23:21:02]

Created Revert of Add lock to fix race in AudioRendererMixerInput.

[commit-bot: I haz the power, 2016-03-04 23:21:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1771463002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1771463002/1

[commit-bot: I haz the power, 2016-03-04 23:22:08]

Description was changed from

==========
Revert of Add lock to fix race in AudioRendererMixerInput. (patchset #5 id:80001
of https://codereview.chromium.org/1748183006/ )

Reason for revert:
This seems to cause failures in LayoutTests/http/tests/security/media. Not sure
why this wasn't caught by CQ... 

Tracking CQ weirdness here:
https://bugs.chromium.org/p/chromium/issues/detail?id=592079


Original issue's description:
> Add lock to fix race in AudioRendererMixerInput.
> 
> Clusterfuzz identified a race between the media thread calling SetVolume
> and the audio device thread calling ProvideInput. Add a lock to
> synchronize access to |volume_| between threads.
> 
> Also adds some thread_checkers to just firm up the contract that the
> majority of these methods are called only by the media thread.
> 
> BUG=588992
> 
> Committed: https://crrev.com/57c6958a99f2c62a4408d2a9d262a22129309106
> Cr-Commit-Position: refs/heads/master@{#379349}

TBR=dalecurtis@chromium.org,olka@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=588992
==========

to

==========
Revert of Add lock to fix race in AudioRendererMixerInput. (patchset #5 id:80001
of https://codereview.chromium.org/1748183006/ )

Reason for revert:
This seems to cause failures in LayoutTests/http/tests/security/media. Not sure
why this wasn't caught by CQ... 

Tracking CQ weirdness here:
https://bugs.chromium.org/p/chromium/issues/detail?id=592079


Original issue's description:
> Add lock to fix race in AudioRendererMixerInput.
> 
> Clusterfuzz identified a race between the media thread calling SetVolume
> and the audio device thread calling ProvideInput. Add a lock to
> synchronize access to |volume_| between threads.
> 
> Also adds some thread_checkers to just firm up the contract that the
> majority of these methods are called only by the media thread.
> 
> BUG=588992
> 
> Committed: https://crrev.com/57c6958a99f2c62a4408d2a9d262a22129309106
> Cr-Commit-Position: refs/heads/master@{#379349}

TBR=dalecurtis@chromium.org,olka@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=588992
==========

[commit-bot: I haz the power, 2016-03-04 23:22:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-04 23:24:03]

Description was changed from

==========
Revert of Add lock to fix race in AudioRendererMixerInput. (patchset #5 id:80001
of https://codereview.chromium.org/1748183006/ )

Reason for revert:
This seems to cause failures in LayoutTests/http/tests/security/media. Not sure
why this wasn't caught by CQ... 

Tracking CQ weirdness here:
https://bugs.chromium.org/p/chromium/issues/detail?id=592079


Original issue's description:
> Add lock to fix race in AudioRendererMixerInput.
> 
> Clusterfuzz identified a race between the media thread calling SetVolume
> and the audio device thread calling ProvideInput. Add a lock to
> synchronize access to |volume_| between threads.
> 
> Also adds some thread_checkers to just firm up the contract that the
> majority of these methods are called only by the media thread.
> 
> BUG=588992
> 
> Committed: https://crrev.com/57c6958a99f2c62a4408d2a9d262a22129309106
> Cr-Commit-Position: refs/heads/master@{#379349}

TBR=dalecurtis@chromium.org,olka@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=588992
==========

to

==========
Revert of Add lock to fix race in AudioRendererMixerInput. (patchset #5 id:80001
of https://codereview.chromium.org/1748183006/ )

Reason for revert:
This seems to cause failures in LayoutTests/http/tests/security/media. Not sure
why this wasn't caught by CQ...

Tracking CQ weirdness here:
https://bugs.chromium.org/p/chromium/issues/detail?id=592079

Original issue's description:
> Add lock to fix race in AudioRendererMixerInput.
>
> Clusterfuzz identified a race between the media thread calling SetVolume
> and the audio device thread calling ProvideInput. Add a lock to
> synchronize access to |volume_| between threads.
>
> Also adds some thread_checkers to just firm up the contract that the
> majority of these methods are called only by the media thread.
>
> BUG=588992
>
> Committed: https://crrev.com/57c6958a99f2c62a4408d2a9d262a22129309106
> Cr-Commit-Position: refs/heads/master@{#379349}

TBR=dalecurtis@chromium.org,olka@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=588992

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

[commit-bot: I haz the power, 2016-03-04 23:24:04]

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

[o1ka, 2016-03-07 16:22:40]

lgtm