WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 4 years ago

#24993 closed defect (bug) (invalid)

Wrong mime type used in <video> tag for .m4v files

Reported by: jrue Owned by:
Milestone: Priority: normal
Severity: major Version: 3.8
Component: Media Keywords:
Focuses: Cc:

Description

In media.php on line 1080, there is no such thing as video/m4v mime type that can be used in the video tag. This essentially breaks all .m4v files from showing up, regardless of the mime type its delivered with. This should be changed to either video/x-m4v (non standard, but Apple recommended) or more appropriately the standards recommended mime video/mp4.

Apple's .m4v files are essentially mp4 encodes wrapped with a different file extension.

​http://tools.ietf.org/html/rfc4337

This is the video I'm testing with.

​http://media.journalism.berkeley.edu/mission/video/20120116_conversion/20120116_conversion-iPhone.m4v

Change History (2)

#1 @wonderboymusic
5 years ago

  • Keywords reporter-feedback added; needs-patch removed

MediaElement checks for video/m4v - the video tag is just a container for whatever ME is going to do to it. I just downloaded your video, it works for me... ​http://cl.ly/image/0r0w0X2C0A17

#2 @wonderboymusic
4 years ago

  • Keywords reporter-feedback removed
  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

See here: ​https://github.com/johndyer/mediaelement/blob/d72ba98927d0d900e5b68f86ec7562da61542747/src/js/me-shim.js#L386

video/m4v is an alias. No other feedback from reporter in 8 months... closing.

Note: See TracTickets for help on using tickets.