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

Unified Diff: media/base/mime_util_internal.cc

Issue 2338213005: Look into full URL spec to see whether it is MP4 (Closed)
Patch Set: Added reference to the bug in the comments. 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
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/base/mime_util_internal.cc
diff --git a/media/base/mime_util_internal.cc b/media/base/mime_util_internal.cc
index 51c2899cc5e6a631c50b715fad361692ac61758b..11a763e74ab3c0054e6a39db8a3bcdbac462b19b 100644
--- a/media/base/mime_util_internal.cc
+++ b/media/base/mime_util_internal.cc
@@ -601,13 +601,9 @@ bool MimeUtil::IsCodecSupportedOnPlatform(
return true;
case H264:
- // The unified pipeline requires platform support for h264.
- if (platform_info.is_unified_media_pipeline_enabled)
- return platform_info.has_platform_decoders;
-
- // When MediaPlayer or MediaCodec is used, h264 is always supported.
- DCHECK(!is_encrypted || platform_info.has_platform_decoders);
- return true;
+ // When content is not encrypted we fall back to MediaPlayer, thus we
+ // always support H264. For EME we need MediaCodec.
+ return !is_encrypted || platform_info.has_platform_decoders;
case HEVC:
#if BUILDFLAG(ENABLE_HEVC_DEMUXING)
« no previous file with comments | « content/renderer/render_frame_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698