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

Unified Diff: media/base/android/java/src/org/chromium/media/AudioManagerAndroid.java

Issue 14049003: Set default sampling rate to 44100 and query the native output sampling rate. (Closed) Base URL: https://src.chromium.org/svn/trunk/src/
Patch Set: rebase Created 7 years, 8 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: media/base/android/java/src/org/chromium/media/AudioManagerAndroid.java
===================================================================
--- media/base/android/java/src/org/chromium/media/AudioManagerAndroid.java (revision 194385)
+++ media/base/android/java/src/org/chromium/media/AudioManagerAndroid.java (working copy)
@@ -18,6 +18,9 @@
@JNINamespace("media")
class AudioManagerAndroid {
private static final String TAG = AudioManagerAndroid.class.getSimpleName();
+ // Most of Google lead devices use 44.1K as the default sampling rate, 44.1K
+ // is also widely used on other android devices.
+ private static final int DEFAULT_SAMPLING_RATE = 44100;
private final AudioManager mAudioManager;
private final Context mContext;
@@ -78,6 +81,18 @@
mAudioManager.setSpeakerphoneOn(mOriginalSpeakerStatus);
}
+ @CalledByNative
+ public int getNativeOutputSampleRate() {
+ if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.JELLY_BEAN_MR1) {
+ String sampleRateString = mAudioManager.getProperty(
qinmin 2013/05/16 01:31:23 nit: No need for 2 if statement: String sampleRat
+ AudioManager.PROPERTY_OUTPUT_SAMPLE_RATE);
+ return (sampleRateString == null ?
+ DEFAULT_SAMPLING_RATE : Integer.parseInt(sampleRateString));
+ } else {
+ return DEFAULT_SAMPLING_RATE;
+ }
+ }
+
private void logDeviceInfo() {
Log.i(TAG, "Manufacturer:" + Build.MANUFACTURER +
" Board: " + Build.BOARD + " Device: " + Build.DEVICE +
« media/audio/android/audio_manager_android.cc ('K') | « media/audio/android/audio_manager_android.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698