Bug 1332568 - MediaMIMETypes can use thread-safe nsContentTypeParser - r?mcmanus draft
authorGerald Squelart <gsquelart@mozilla.com>
Fri, 20 Jan 2017 17:17:27 +1100
changeset 468160 b07e2555dad4194ce64c8b75a275a883977ff80d
parent 468159 9418f3c879846a75975bd40bc9b3e88894f23c49
child 543869 e2231d2fba23b96f74cc30963808a445da36e742
push id43375
push usergsquelart@mozilla.com
push dateMon, 30 Jan 2017 20:45:20 +0000
reviewersmcmanus
bugs1332568
milestone54.0a1
Bug 1332568 - MediaMIMETypes can use thread-safe nsContentTypeParser - r?mcmanus MozReview-Commit-ID: 7Gpwx2SSVba
dom/media/MediaMIMETypes.cpp
--- a/dom/media/MediaMIMETypes.cpp
+++ b/dom/media/MediaMIMETypes.cpp
@@ -47,18 +47,16 @@ MediaMIMEType::SizeOfExcludingThis(Mallo
 MediaMIMEType::MediaMIMEType(const nsACString& aType)
   : mMIMEType(aType)
 {
 }
 
 Maybe<MediaMIMEType>
 MakeMediaMIMEType(const nsAString& aType)
 {
-  MOZ_ASSERT(NS_IsMainThread());
-
   nsContentTypeParser parser(aType);
   nsAutoString mime;
   nsresult rv = parser.GetType(mime);
   if (!NS_SUCCEEDED(rv) || mime.IsEmpty()) {
     return Nothing();
   }
 
   NS_ConvertUTF16toUTF8 mime8{mime};
@@ -158,18 +156,16 @@ MediaExtendedMIMEType::MediaExtendedMIME
   : mOriginalString(aType.AsString())
   , mMIMEType(Move(aType))
 {
 }
 
 Maybe<MediaExtendedMIMEType>
 MakeMediaExtendedMIMEType(const nsAString& aType)
 {
-  MOZ_ASSERT(NS_IsMainThread());
-
   nsContentTypeParser parser(aType);
   nsAutoString mime;
   nsresult rv = parser.GetType(mime);
   if (!NS_SUCCEEDED(rv) || mime.IsEmpty()) {
     return Nothing();
   }
 
   NS_ConvertUTF16toUTF8 mime8{mime};