[wp-trac] [WordPress Trac] #40921: Inconsistent Handling of mp4 by Audio Widget

[WordPress Trac]

#40921: Inconsistent Handling of mp4 by Audio Widget
-------------------------------------------------+-------------------------
 Reporter:  toddhalfpenny                        |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  Future
Component:  Media                                |  Release
 Severity:  normal                               |     Version:  3.6
 Keywords:  2nd-opinion dev-feedback has-        |  Resolution:
  screenshots has-patch needs-unit-tests         |     Focuses:
-------------------------------------------------+-------------------------

Comment (by blobfolio):

 Replying to [comment:29 westonruter]:
 > Wouldn't checking the file like this be a problem for performance?

 This patch does result in an extra call to the `getid3.php` library for
 files named *.mp4 or *.ogg, but the overhead from that library is not a
 very significant percentage of the overall request. Multimedia files might
 be on the larger side, but their metadata is small, and that's all that
 actually ends up getting read into memory (mostly).

 I ran some quick tests on a fresh `twentyseventeen` site, and it seems
 that uploading a regular image file takes about 1.5x longer to process.

 Not that there isn't room for improvement. :)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/40921#comment:30>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform