Bug 1097126 - Restrict MediaCodec backend to Android 5.0 and higher r=blassey a=lmandel
💩💩 backed out by b0fea8a116aa 💩 💩
authorJames Willcox <snorp@snorp.net>
Tue, 11 Nov 2014 15:04:36 -0600
changeset 226026 7dfbe52d1a2b
parent 226025 e7f8aa528841
child 226027 b0fea8a116aa
push id4112
push userjwillcox@mozilla.com
push date2014-11-11 21:05 +0000
treeherdermozilla-beta@7dfbe52d1a2b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersblassey, lmandel
bugs1097126
milestone34.0
Bug 1097126 - Restrict MediaCodec backend to Android 5.0 and higher r=blassey a=lmandel
content/media/fmp4/MP4Decoder.cpp
--- a/content/media/fmp4/MP4Decoder.cpp
+++ b/content/media/fmp4/MP4Decoder.cpp
@@ -178,18 +178,18 @@ static bool
 HavePlatformMPEGDecoders()
 {
   return Preferences::GetBool("media.fragmented-mp4.use-blank-decoder") ||
 #ifdef XP_WIN
          // We have H.264/AAC platform decoders on Windows Vista and up.
          IsVistaOrLater() ||
 #endif
 #ifdef MOZ_WIDGET_ANDROID
-         // We need android.media.MediaCodec which exists in API level 16 and higher.
-         (AndroidBridge::Bridge()->GetAPIVersion() >= 16) ||
+         // Works on 16 and higher, but restrict to 21 (Lollipop) and higher
+         (AndroidBridge::Bridge()->GetAPIVersion() >= 21) ||
 #endif
          IsFFmpegAvailable() ||
          IsAppleAvailable() ||
          IsGonkMP4DecoderAvailable() ||
          // TODO: Other platforms...
          false;
 }