corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1289668 - Refactor FrameStatistics - r=kamidphish

Browse Source 
Move FrameStatistics' data into separate struct, so that it can more easily be
changed and passed around, outside of the lock-controlled FrameStatistics
object.

MozReview-Commit-ID: TfsMRJhVfQ
This commit is contained in:
Gerald Squelart
2016-07-18 09:51:25 +10:00
parent dda21065e1
commit ba51d3e4a1
2 changed files with 41 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
dom/html/HTMLVideoElement.cpp
View File

@@ -242,11 +242,12 @@ HTMLVideoElement::GetVideoPlaybackQuality()
}
if (mDecoder) {
FrameStatistics& stats = mDecoder->GetFrameStatistics();
static_assert(sizeof(uint32_t) >= sizeof (stats.GetParsedFrames()),
FrameStatisticsData stats =
mDecoder->GetFrameStatistics().GetFrameStatisticsData();
static_assert(sizeof(totalFrames) >= sizeof(stats.mParsedFrames),
"possible truncation from FrameStatistics to VideoPlaybackQuality");
totalFrames = stats.GetParsedFrames();
droppedFrames = stats.GetDroppedFrames();
totalFrames = stats.mParsedFrames;
droppedFrames = stats.mDroppedFrames;
corruptedFrames = 0;
}
}