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

Unified Diff: media/audio/sample_rates.h

Issue 10914203: Add UMA reporting for audio hardware parameters. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Comments. Created 8 years, 3 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
« no previous file with comments | « media/audio/audio_output_resampler.cc ('k') | media/audio/sample_rates.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/audio/sample_rates.h
diff --git a/media/audio/sample_rates.h b/media/audio/sample_rates.h
index 9d47a138bc2afd6784478aa14ba4b5daf5b4f91d..7c29e548b340e385512eedf82821a0f9f15f7060 100644
--- a/media/audio/sample_rates.h
+++ b/media/audio/sample_rates.h
@@ -10,8 +10,8 @@
namespace media {
// Enumeration used for histogramming sample rates into distinct buckets.
+// Logged to UMA, so never reuse a value, always add new/greater ones!
enum AudioSampleRate {
- // Do not change the order of these values.
k8000Hz = 0,
k16000Hz = 1,
k32000Hz = 2,
@@ -20,6 +20,9 @@ enum AudioSampleRate {
k11025Hz = 5,
k22050Hz = 6,
k44100Hz = 7,
+ k88200Hz = 8,
+ k176400Hz = 9,
+ k192000Hz = 10,
kUnexpectedAudioSampleRate // Must always be last!
};
« no previous file with comments | « media/audio/audio_output_resampler.cc ('k') | media/audio/sample_rates.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698