Bug 1115594: Fix indentation in AppleVDADecoder. r=rillian
authorJean-Yves Avenard <jyavenard@mozilla.com>
Tue, 30 Dec 2014 12:06:22 +1100
changeset 247455 0cb97323a1bd9952e5deef03cc9fc55d00daa435
parent 247454 f1fb6e0260a86ac5d976259b74e5740c7b3c45be
child 247456 2f51262d2cabf17455ae7bdfc0ae69db10f031f6
push id4489
push userraliiev@mozilla.com
push dateMon, 23 Feb 2015 15:17:55 +0000
treeherdermozilla-beta@fd7c3dc24146 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrillian
bugs1115594
milestone37.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1115594: Fix indentation in AppleVDADecoder. r=rillian
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