Skip to content

Commit

Permalink
Remove the call to Terminate() on the main thread.
Browse files Browse the repository at this point in the history
Terminate() must be called on the signaling thread so if the WebRtcAudioDevice hasn't already been uninitialized, there isn't much we can do and instead we need to figure out why and fix it (there might not be a problem).

BUG=447042
R=henrika@chromium.org

Review URL: https://codereview.chromium.org/1523053006 .

Cr-Commit-Position: refs/heads/master@{#365218}
  • Loading branch information
Tommi committed Dec 15, 2015
1 parent e9e76ec commit 8a5db5a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion content/renderer/media/webrtc_audio_device_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ WebRtcAudioDeviceImpl::WebRtcAudioDeviceImpl()
WebRtcAudioDeviceImpl::~WebRtcAudioDeviceImpl() {
DVLOG(1) << "WebRtcAudioDeviceImpl::~WebRtcAudioDeviceImpl()";
DCHECK(main_thread_checker_.CalledOnValidThread());
Terminate();
DCHECK(!initialized_) << "Terminate must have been called.";
}

int32_t WebRtcAudioDeviceImpl::AddRef() const {
Expand Down

0 comments on commit 8a5db5a

Please sign in to comment.