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

Unified Diff: content/browser/renderer_host/media/audio_input_renderer_host.cc

Issue 2624403002: Refactor AudioInputController and related interfaces. (Closed)
Patch Set: Address comments Created 3 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/media/audio_input_renderer_host.cc
diff --git a/content/browser/renderer_host/media/audio_input_renderer_host.cc b/content/browser/renderer_host/media/audio_input_renderer_host.cc
index d0c9b327578574dc98058b9eee2069f60a36d7ec..9bb4969ee600f91a51df54c512a49d08944b63bb 100644
--- a/content/browser/renderer_host/media/audio_input_renderer_host.cc
+++ b/content/browser/renderer_host/media/audio_input_renderer_host.cc
@@ -156,11 +156,6 @@ void AudioInputRendererHost::OnError(media::AudioInputController* controller,
base::RetainedRef(controller), error_code));
}
-void AudioInputRendererHost::OnData(media::AudioInputController* controller,
- const media::AudioBus* data) {
- NOTREACHED() << "Only low-latency mode is supported.";
-}
-
void AudioInputRendererHost::OnLog(media::AudioInputController* controller,
const std::string& message) {
BrowserThread::PostTask(BrowserThread::IO, FROM_HERE,
@@ -180,7 +175,6 @@ void AudioInputRendererHost::DoCompleteCreation(
AudioEntry* entry = LookupByController(controller);
DCHECK(entry);
DCHECK(PeerHandle());
- DCHECK(entry->controller->SharedMemoryAndSyncSocketMode());
// Once the audio stream is created then complete the creation process by
// mapping shared memory and sharing with the renderer process.

Powered by Google App Engine
This is Rietveld 408576698