Bug 1115594 - Fix indentation in AppleVDADecoder. r=rillian, a=sledru
authorJean-Yves Avenard <jyavenard@mozilla.com>
Tue, 30 Dec 2014 12:06:22 +1100
changeset 242740 1aaa04637144d5775c2b6a9fb5915aecf007c84a
parent 242739 b777575191d95185e8b822c8632edfbcfd1de034
child 242741 7318fa1f151cc529b18ed6a2f200883ea02e34c6
push id4311
push userraliiev@mozilla.com
push dateMon, 12 Jan 2015 19:37:41 +0000
treeherdermozilla-beta@150c9fed433b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrillian, sledru
bugs1115594
milestone36.0a2
Bug 1115594 - Fix indentation in AppleVDADecoder. r=rillian, a=sledru
dom/media/fmp4/apple/AppleVDADecoder.cpp
--- a/dom/media/fmp4/apple/AppleVDADecoder.cpp
+++ b/dom/media/fmp4/apple/AppleVDADecoder.cpp
@@ -284,20 +284,18 @@ AppleVDADecoder::OutputFrame(CVPixelBuff
     NS_ERROR("Couldn't create VideoData for frame");
     mCallback->Error();
     return NS_ERROR_FAILURE;
   }
 
   // Frames come out in DTS order but we need to output them
   // in composition order.
   mReorderQueue.Push(data);
-  // Assume a frame with a PTS <= current DTS is ready.
   while (mReorderQueue.Length() > mMaxRefFrames) {
-    nsRefPtr<VideoData> readyData = mReorderQueue.Pop();
-      mCallback->Output(readyData);
+    mCallback->Output(mReorderQueue.Pop());
   }
   LOG("%llu decoded frames queued",
       static_cast<unsigned long long>(mReorderQueue.Length()));
 
   return NS_OK;
 }
 
 nsresult