Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(809)

Side by Side Diff: content/browser/renderer_host/media/audio_input_renderer_host.cc

Issue 1005683003: favor DCHECK_CURRENTLY_ON for better logs in content/browser/[q-z]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/renderer_host/media/audio_input_renderer_host.h" 5 #include "content/browser/renderer_host/media/audio_input_renderer_host.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/memory/shared_memory.h" 8 #include "base/memory/shared_memory.h"
9 #include "base/metrics/histogram.h" 9 #include "base/metrics/histogram.h"
10 #include "base/numerics/safe_math.h" 10 #include "base/numerics/safe_math.h"
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 media::UserInputMonitor* user_input_monitor) 77 media::UserInputMonitor* user_input_monitor)
78 : BrowserMessageFilter(AudioMsgStart), 78 : BrowserMessageFilter(AudioMsgStart),
79 audio_manager_(audio_manager), 79 audio_manager_(audio_manager),
80 media_stream_manager_(media_stream_manager), 80 media_stream_manager_(media_stream_manager),
81 audio_mirroring_manager_(audio_mirroring_manager), 81 audio_mirroring_manager_(audio_mirroring_manager),
82 user_input_monitor_(user_input_monitor), 82 user_input_monitor_(user_input_monitor),
83 audio_log_(MediaInternals::GetInstance()->CreateAudioLog( 83 audio_log_(MediaInternals::GetInstance()->CreateAudioLog(
84 media::AudioLogFactory::AUDIO_INPUT_CONTROLLER)) {} 84 media::AudioLogFactory::AUDIO_INPUT_CONTROLLER)) {}
85 85
86 AudioInputRendererHost::~AudioInputRendererHost() { 86 AudioInputRendererHost::~AudioInputRendererHost() {
87 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO)); 87 DCHECK_CURRENTLY_ON(BrowserThread::IO);
88 DCHECK(audio_entries_.empty()); 88 DCHECK(audio_entries_.empty());
89 } 89 }
90 90
91 void AudioInputRendererHost::OnChannelClosing() { 91 void AudioInputRendererHost::OnChannelClosing() {
92 // Since the IPC sender is gone, close all requested audio streams. 92 // Since the IPC sender is gone, close all requested audio streams.
93 DeleteEntries(); 93 DeleteEntries();
94 } 94 }
95 95
96 void AudioInputRendererHost::OnDestruct() const { 96 void AudioInputRendererHost::OnDestruct() const {
97 BrowserThread::DeleteOnIOThread::Destruct(this); 97 BrowserThread::DeleteOnIOThread::Destruct(this);
(...skipping 452 matching lines...) Expand 10 before | Expand all | Expand 10 after
550 #if defined(OS_CHROMEOS) 550 #if defined(OS_CHROMEOS)
551 if (config.params.channel_layout() == 551 if (config.params.channel_layout() ==
552 media::CHANNEL_LAYOUT_STEREO_AND_KEYBOARD_MIC) { 552 media::CHANNEL_LAYOUT_STEREO_AND_KEYBOARD_MIC) {
553 media_stream_manager_->audio_input_device_manager() 553 media_stream_manager_->audio_input_device_manager()
554 ->UnregisterKeyboardMicStream(); 554 ->UnregisterKeyboardMicStream();
555 } 555 }
556 #endif 556 #endif
557 } 557 }
558 558
559 } // namespace content 559 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/renderer_data_memoizing_store.h ('k') | content/browser/renderer_host/media/media_stream_ui_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698