summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorhenrika <henrika@chromium.org>2014-12-08 14:47:45 -0800
committerCommit bot <commit-bot@chromium.org>2014-12-08 22:48:17 +0000
commitf40606db534e35aa8d1a54a45f39abd38ba57734 (patch)
treee8f13bf7f8fdddcf145636be7da967c03def1ab3
parentdd9333182b7e6d3eaff0bf6937f6b3ef14a53d49 (diff)
downloadchromium_src-f40606db534e35aa8d1a54a45f39abd38ba57734.zip
chromium_src-f40606db534e35aa8d1a54a45f39abd38ba57734.tar.gz
chromium_src-f40606db534e35aa8d1a54a45f39abd38ba57734.tar.bz2
Add Media.FallbackToHighLatencyAudioInputPath.
Adds UMA stat to count the number of times we fail to create a low-latency audio input stream on Windows. BUG=411380 Review URL: https://codereview.chromium.org/769473002 Cr-Commit-Position: refs/heads/master@{#307363}
-rw-r--r--media/audio/win/audio_manager_win.cc2
-rw-r--r--tools/metrics/histograms/histograms.xml8
2 files changed, 10 insertions, 0 deletions
diff --git a/media/audio/win/audio_manager_win.cc b/media/audio/win/audio_manager_win.cc
index 4a1bc747..5d9d623 100644
--- a/media/audio/win/audio_manager_win.cc
+++ b/media/audio/win/audio_manager_win.cc
@@ -16,6 +16,7 @@
#include "base/files/file_path.h"
#include "base/memory/scoped_ptr.h"
#include "base/message_loop/message_loop.h"
+#include "base/metrics/histogram.h"
#include "base/path_service.h"
#include "base/process/launch.h"
#include "base/strings/string_number_conversions.h"
@@ -393,6 +394,7 @@ AudioInputStream* AudioManagerWin::MakeLowLatencyInputStream(
DCHECK_EQ(AudioParameters::AUDIO_PCM_LOW_LATENCY, params.format());
DVLOG(1) << "MakeLowLatencyInputStream: " << device_id;
AudioInputStream* stream = NULL;
+ UMA_HISTOGRAM_BOOLEAN("Media.WindowsCoreAudioInput", core_audio_supported());
if (!core_audio_supported()) {
// Fall back to Windows Wave implementation on Windows XP or lower.
DVLOG(1) << "Using WaveIn since WASAPI requires at least Vista.";
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index f656ffb..91eaf62 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -12852,6 +12852,14 @@ Therefore, the affected-histogram name has to have at least one dot in it.
</summary>
</histogram>
+<histogram name="Media.WindowsCoreAudioInput" enum="BooleanSuccess">
+ <owner>henrika@chromium.org</owner>
+ <summary>
+ Whether Chrome is using Windows Core Audio for audio input or not. Updated
+ on Windows only when a low-latency audio input stream is created.
+ </summary>
+</histogram>
+
<histogram name="Media.YouTube.DelayedAndDroppedFramesPer5Sec"
units="frames/5s">
<owner>Please list the metric's owners. Add more owner tags as needed.</owner>