diff options
| author | chcunningham <chcunningham@chromium.org> | 2016-03-04 15:22:01 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-03-04 23:23:54 +0000 |
| commit | 6b206d7b4963be6e8e7ff269dcf0015eb7d8bffa (patch) | |
| tree | ea7cfc69d2a9172ddd43cfbff9131061e3deeda5 /ipc/handle_attachment_win.cc | |
| parent | b5421b5647a0627226b8b2f48e6143b9d0cf3647 (diff) | |
| download | chromium_src-6b206d7b4963be6e8e7ff269dcf0015eb7d8bffa.zip chromium_src-6b206d7b4963be6e8e7ff269dcf0015eb7d8bffa.tar.gz chromium_src-6b206d7b4963be6e8e7ff269dcf0015eb7d8bffa.tar.bz2 | |
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
Review URL: https://codereview.chromium.org/1771463002
Cr-Commit-Position: refs/heads/master@{#379388}
Diffstat (limited to 'ipc/handle_attachment_win.cc')
0 files changed, 0 insertions, 0 deletions