Bug 1396629 - P1. Use nsCString. r?cpearce draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Mon, 04 Sep 2017 21:07:11 +0200
changeset 658802 e84de159ee1bf6ae95358ef44742d0a5e5ecf787
parent 658734 6e7b95decca7c19e32e00abf1d84f35f96cab868
child 658803 7fc7926dc4562a9dfb1d73e4cf6d84ca82c9b004
push id77874
push userbmo:jyavenard@mozilla.com
push dateMon, 04 Sep 2017 21:38:59 +0000
reviewerscpearce
bugs1396629
milestone57.0a1
Bug 1396629 - P1. Use nsCString. r?cpearce The default string is over 400 bytes long, it will never fit in the default 64 bytes buffer of an nsAutoCString. MozReview-Commit-ID: 3FHPQDgCtMF
dom/media/MediaFormatReader.cpp
--- a/dom/media/MediaFormatReader.cpp
+++ b/dom/media/MediaFormatReader.cpp
@@ -3112,17 +3112,17 @@ MediaFormatReader::GetImageContainer()
 {
   return mVideoFrameContainer ? mVideoFrameContainer->GetImageContainer()
                               : nullptr;
 }
 
 void
 MediaFormatReader::GetMozDebugReaderData(nsACString& aString)
 {
-  nsAutoCString result;
+  nsCString result;
   nsAutoCString audioDecoderName("unavailable");
   nsAutoCString videoDecoderName = audioDecoderName;
   nsAutoCString audioType("none");
   nsAutoCString videoType("none");
 
   if (HasAudio()) {
     MutexAutoLock lock(mAudio.mMutex);
     audioDecoderName = mAudio.mDescription;