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

Unified Diff: media/audio/android/audio_manager_android.cc

Issue 2379303003: media: Delete code checking for pre-JellyBean Android versions (Closed)
Patch Set: Created 4 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
Index: media/audio/android/audio_manager_android.cc
diff --git a/media/audio/android/audio_manager_android.cc b/media/audio/android/audio_manager_android.cc
index c8fa1761f09ed1eec19e12ad45428fb2df4d2373..b6d339abf93b10def078fdc63942dd9d375db1c8 100644
--- a/media/audio/android/audio_manager_android.cc
+++ b/media/audio/android/audio_manager_android.cc
@@ -263,12 +263,7 @@ AudioInputStream* AudioManagerAndroid::MakeLowLatencyInputStream(
if (params.effects() != AudioParameters::NO_EFFECTS) {
// Platform effects can only be enabled through the AudioRecord path.
// An effect should only have been requested here if recommended by
- // AudioManagerAndroid.shouldUse<Effect>.
- //
- // Creating this class requires Jelly Bean, which is already guaranteed by
- // shouldUse<Effect>. Only DCHECK on that condition to allow tests to use
- // the effect settings as a way to select the input path.
- DCHECK_GE(base::android::BuildInfo::GetInstance()->sdk_int(), 16);
+ // AudioManagerAndroid#shouldUse...().
DVLOG(1) << "Creating AudioRecordInputStream";
return new AudioRecordInputStream(this, params);
}

Powered by Google App Engine
This is Rietveld 408576698