Bug 1237160: Do not count frames not composited as dropped. r=cpearce

This is a partial reversal of bug 1230338. We can't distinguish frames that are never composited because the media element is hidden from the frames genuinely dropped due to machine slowness. So until we can distinguish them, let's not report them as dropped.

MozReview-Commit-ID: ERV8Luaxp3F
This commit is contained in:
Jean-Yves Avenard 2016-02-03 18:07:44 +11:00
parent a881759b48
commit 41241eef26
2 changed files with 0 additions and 7 deletions

View File

@ -34,7 +34,6 @@ VideoSink::VideoSink(AbstractThread* aThread,
, mProducerID(ImageContainer::AllocateProducerID())
, mFrameStats(aFrameStats)
, mVideoFrameEndTime(-1)
, mOldDroppedCount(0)
, mHasVideo(false)
, mUpdateScheduler(aThread)
, mVideoQueueSendToCompositorSize(aVQueueSentToCompositerSize)
@ -364,10 +363,6 @@ VideoSink::RenderVideoFrames(int32_t aMaxFrames,
frame->mTime, frame->mFrameID, VideoQueue().GetSize());
}
mContainer->SetCurrentFrames(frames[0]->As<VideoData>()->mDisplay, images);
uint32_t dropped = mContainer->GetDroppedImageCount();
mFrameStats.NotifyDecodedFrames(0, 0, dropped - mOldDroppedCount);
mOldDroppedCount = dropped;
}
void

View File

@ -126,8 +126,6 @@ private:
// in microseconds.
int64_t mVideoFrameEndTime;
uint32_t mOldDroppedCount;
// Event listeners for VideoQueue
MediaEventListener mPushListener;
MediaEventListener mFinishListener;