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

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

Issue 2702323002: Move AudioDebugFileWriter from content/ to media/. (Closed)
Patch Set: Code review, unit test fix and rebase. Created 3 years, 10 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 <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
11 #include "base/files/file.h" 11 #include "base/files/file.h"
12 #include "base/memory/ref_counted.h" 12 #include "base/memory/ref_counted.h"
13 #include "base/memory/shared_memory.h" 13 #include "base/memory/shared_memory.h"
14 #include "base/metrics/histogram_macros.h" 14 #include "base/metrics/histogram_macros.h"
15 #include "base/numerics/safe_math.h" 15 #include "base/numerics/safe_math.h"
16 #include "base/process/process.h" 16 #include "base/process/process.h"
17 #include "base/strings/string_number_conversions.h" 17 #include "base/strings/string_number_conversions.h"
18 #include "base/strings/stringprintf.h" 18 #include "base/strings/stringprintf.h"
19 #include "base/sync_socket.h" 19 #include "base/sync_socket.h"
20 #include "build/build_config.h" 20 #include "build/build_config.h"
21 #include "content/browser/bad_message.h" 21 #include "content/browser/bad_message.h"
22 #include "content/browser/media/capture/desktop_capture_device_uma_types.h" 22 #include "content/browser/media/capture/desktop_capture_device_uma_types.h"
23 #include "content/browser/media/capture/web_contents_audio_input_stream.h" 23 #include "content/browser/media/capture/web_contents_audio_input_stream.h"
24 #include "content/browser/media/media_internals.h" 24 #include "content/browser/media/media_internals.h"
25 #include "content/browser/renderer_host/media/audio_debug_file_writer.h"
26 #include "content/browser/renderer_host/media/audio_input_device_manager.h" 25 #include "content/browser/renderer_host/media/audio_input_device_manager.h"
27 #include "content/browser/renderer_host/media/audio_input_sync_writer.h" 26 #include "content/browser/renderer_host/media/audio_input_sync_writer.h"
28 #include "content/browser/renderer_host/media/media_stream_manager.h" 27 #include "content/browser/renderer_host/media/media_stream_manager.h"
29 #include "content/browser/webrtc/webrtc_internals.h" 28 #include "content/browser/webrtc/webrtc_internals.h"
30 #include "content/public/browser/web_contents_media_capture_id.h" 29 #include "content/public/browser/web_contents_media_capture_id.h"
31 #include "media/audio/audio_device_description.h" 30 #include "media/audio/audio_device_description.h"
32 #include "media/base/audio_bus.h" 31 #include "media/base/audio_bus.h"
33 #include "media/base/media_switches.h" 32 #include "media/base/media_switches.h"
34 #include "media/media_features.h" 33 #include "media/media_features.h"
35 34
(...skipping 298 matching lines...) Expand 10 before | Expand all | Expand 10 after
334 std::unique_ptr<AudioInputSyncWriter> writer(new AudioInputSyncWriter( 333 std::unique_ptr<AudioInputSyncWriter> writer(new AudioInputSyncWriter(
335 entry->shared_memory.memory(), entry->shared_memory.requested_size(), 334 entry->shared_memory.memory(), entry->shared_memory.requested_size(),
336 entry->shared_memory_segment_count, audio_params)); 335 entry->shared_memory_segment_count, audio_params));
337 336
338 if (!writer->Init()) { 337 if (!writer->Init()) {
339 SendErrorMessage(stream_id, SYNC_WRITER_INIT_FAILED); 338 SendErrorMessage(stream_id, SYNC_WRITER_INIT_FAILED);
340 MaybeUnregisterKeyboardMicStream(config); 339 MaybeUnregisterKeyboardMicStream(config);
341 return; 340 return;
342 } 341 }
343 342
344 #if BUILDFLAG(ENABLE_WEBRTC)
345 std::unique_ptr<media::AudioFileWriter> debug_writer(
346 new AudioDebugFileWriter(audio_params));
347 #else
348 std::unique_ptr<media::AudioFileWriter> debug_writer(nullptr);
349 #endif
350
351 // If we have successfully created the SyncWriter then assign it to the 343 // If we have successfully created the SyncWriter then assign it to the
352 // entry and construct an AudioInputController. 344 // entry and construct an AudioInputController.
353 entry->writer.reset(writer.release()); 345 entry->writer.reset(writer.release());
354 if (WebContentsMediaCaptureId::Parse(device_id, nullptr)) { 346 if (WebContentsMediaCaptureId::Parse(device_id, nullptr)) {
355 // For MEDIA_DESKTOP_AUDIO_CAPTURE, the source is selected from picker 347 // For MEDIA_DESKTOP_AUDIO_CAPTURE, the source is selected from picker
356 // window, we do not mute the source audio. 348 // window, we do not mute the source audio.
357 // For MEDIA_TAB_AUDIO_CAPTURE, the probable use case is Cast, we mute 349 // For MEDIA_TAB_AUDIO_CAPTURE, the probable use case is Cast, we mute
358 // the source audio. 350 // the source audio.
359 // TODO(qiangchen): Analyze audio constraints to make a duplicating or 351 // TODO(qiangchen): Analyze audio constraints to make a duplicating or
360 // diverting decision. It would give web developer more flexibility. 352 // diverting decision. It would give web developer more flexibility.
361 entry->controller = media::AudioInputController::CreateForStream( 353 entry->controller = media::AudioInputController::CreateForStream(
362 audio_manager_->GetTaskRunner(), this, 354 audio_manager_->GetTaskRunner(), this,
363 WebContentsAudioInputStream::Create( 355 WebContentsAudioInputStream::Create(
364 device_id, audio_params, audio_manager_->GetWorkerTaskRunner(), 356 device_id, audio_params, audio_manager_->GetWorkerTaskRunner(),
365 audio_mirroring_manager_), 357 audio_mirroring_manager_),
366 entry->writer.get(), std::move(debug_writer), user_input_monitor_); 358 entry->writer.get(), user_input_monitor_,
359 BrowserThread::GetTaskRunnerForThread(BrowserThread::FILE),
360 audio_params);
367 // Only count for captures from desktop media picker dialog. 361 // Only count for captures from desktop media picker dialog.
368 if (entry->controller.get() && type == MEDIA_DESKTOP_AUDIO_CAPTURE) 362 if (entry->controller.get() && type == MEDIA_DESKTOP_AUDIO_CAPTURE)
369 IncrementDesktopCaptureCounter(TAB_AUDIO_CAPTURER_CREATED); 363 IncrementDesktopCaptureCounter(TAB_AUDIO_CAPTURER_CREATED);
370 } else { 364 } else {
371 entry->controller = media::AudioInputController::Create( 365 entry->controller = media::AudioInputController::Create(
372 audio_manager_, this, entry->writer.get(), user_input_monitor_, 366 audio_manager_, this, entry->writer.get(), user_input_monitor_,
373 std::move(debug_writer), audio_params, device_id, 367 audio_params, device_id, config.automatic_gain_control,
374 config.automatic_gain_control); 368 BrowserThread::GetTaskRunnerForThread(BrowserThread::FILE));
375 oss << ", AGC=" << config.automatic_gain_control; 369 oss << ", AGC=" << config.automatic_gain_control;
376 370
377 // Only count for captures from desktop media picker dialog and system loop 371 // Only count for captures from desktop media picker dialog and system loop
378 // back audio. 372 // back audio.
379 if (entry->controller.get() && type == MEDIA_DESKTOP_AUDIO_CAPTURE && 373 if (entry->controller.get() && type == MEDIA_DESKTOP_AUDIO_CAPTURE &&
380 (device_id == media::AudioDeviceDescription::kLoopbackInputDeviceId || 374 (device_id == media::AudioDeviceDescription::kLoopbackInputDeviceId ||
381 device_id == 375 device_id ==
382 media::AudioDeviceDescription::kLoopbackWithMuteDeviceId)) { 376 media::AudioDeviceDescription::kLoopbackWithMuteDeviceId)) {
383 IncrementDesktopCaptureCounter(SYSTEM_LOOPBACK_AUDIO_CAPTURER_CREATED); 377 IncrementDesktopCaptureCounter(SYSTEM_LOOPBACK_AUDIO_CAPTURER_CREATED);
384 } 378 }
(...skipping 210 matching lines...) Expand 10 before | Expand all | Expand 10 after
595 entry->controller->EnableDebugRecording( 589 entry->controller->EnableDebugRecording(
596 file_name.AddExtension(IntToStringType(stream_id)) 590 file_name.AddExtension(IntToStringType(stream_id))
597 .AddExtension(kDebugRecordingFileNameExtension)); 591 .AddExtension(kDebugRecordingFileNameExtension));
598 } 592 }
599 593
600 #undef IntToStringType 594 #undef IntToStringType
601 595
602 #endif // BUILDFLAG(ENABLE_WEBRTC) 596 #endif // BUILDFLAG(ENABLE_WEBRTC)
603 597
604 } // namespace content 598 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698