Bug 1251184: [quicktime] P1. Report video/quicktime mimetype when sniffing. r?cpearce draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Fri, 11 Mar 2016 22:32:30 +1100
changeset 339476 b7459a5de4b2100b0296b395afc671d3d6cbe5d8
parent 339422 4dccba6aa5a2dc57f24112edd57295bc880a9e33
child 339477 4a4c4fab81ba65722ae03f37a1199a249a677f58
push id12741
push userbmo:jyavenard@mozilla.com
push dateFri, 11 Mar 2016 11:42:24 +0000
reviewerscpearce
bugs1251184
milestone48.0a1
Bug 1251184: [quicktime] P1. Report video/quicktime mimetype when sniffing. r?cpearce MozReview-Commit-ID: 9EJQVYt76Mh
netwerk/mime/nsMimeTypes.h
toolkit/components/mediasniffer/nsMediaSniffer.cpp
--- a/netwerk/mime/nsMimeTypes.h
+++ b/netwerk/mime/nsMimeTypes.h
@@ -146,16 +146,17 @@
 #define TEXT_ECMASCRIPT                     "text/ecmascript"
 #define TEXT_JAVASCRIPT                     "text/javascript"
 #define TEXT_XSL                            "text/xsl"
 #define TEXT_EVENT_STREAM                   "text/event-stream"
 #define TEXT_CACHE_MANIFEST                 "text/cache-manifest"
 
 #define VIDEO_MPEG                          "video/mpeg"
 #define VIDEO_MP4                           "video/mp4"
+#define VIDEO_QUICKTIME                     "video/quicktime"
 #define VIDEO_RAW                           "video/x-raw-yuv"
 #define VIDEO_OGG                           "video/ogg"
 #define VIDEO_WEBM                          "video/webm"
 #define VIDEO_3GPP                          "video/3gpp"
 #define VIDEO_3GPP2                         "video/3gpp2"
 #define VIDEO_MPEG_TS                       "video/mp2t"
 #define VIDEO_AVI                           "video/avi"
 #define VIDEO_MATROSKA                      "video/x-matroska"
--- a/toolkit/components/mediasniffer/nsMediaSniffer.cpp
+++ b/toolkit/components/mediasniffer/nsMediaSniffer.cpp
@@ -38,17 +38,17 @@ nsMediaSnifferEntry nsMediaSniffer::sSni
 
 // For a complete list of file types, see http://www.ftyps.com/index.html
 nsMediaSnifferEntry sFtypEntries[] = {
   PATTERN_ENTRY("\xFF\xFF\xFF", "mp4", VIDEO_MP4), // Could be mp41 or mp42.
   PATTERN_ENTRY("\xFF\xFF\xFF", "avc", VIDEO_MP4), // Could be avc1, avc2, ...
   PATTERN_ENTRY("\xFF\xFF\xFF", "3gp", VIDEO_3GPP), // Could be 3gp4, 3gp5, ...
   PATTERN_ENTRY("\xFF\xFF\xFF\xFF", "M4A ", AUDIO_MP4),
   PATTERN_ENTRY("\xFF\xFF\xFF\xFF", "M4P ", AUDIO_MP4),
-  PATTERN_ENTRY("\xFF\xFF\xFF\xFF", "qt  ", VIDEO_MP4),
+  PATTERN_ENTRY("\xFF\xFF\xFF\xFF", "qt  ", VIDEO_QUICKTIME),
 };
 
 static bool MatchesBrands(const uint8_t aData[4], nsACString& aSniffedType)
 {
   for (size_t i = 0; i < mozilla::ArrayLength(sFtypEntries); ++i) {
     const auto& currentEntry = sFtypEntries[i];
     bool matched = true;
     MOZ_ASSERT(currentEntry.mLength <= 4, "Pattern is too large to match brand strings.");