Bug 1109644. Part 4: Make MediaEngineGonkVideoSource use GetEndOfAppendedData instead of aLastEndTime. r=jesup
authorRobert O'Callahan <robert@ocallahan.org>
Tue, 30 Dec 2014 14:54:02 +1300
changeset 247536 412b935dc4a8799c8bf8f3bfd091f919eb6f8af6
parent 247535 2199fa1fc9cb7e67c4aefcc348774c5edb70f3bc
child 247537 080bbd228f63ea33a13d6b8e5012373cd3536bfc
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)
reviewersjesup
bugs1109644
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 1109644. Part 4: Make MediaEngineGonkVideoSource use GetEndOfAppendedData instead of aLastEndTime. r=jesup
dom/media/webrtc/MediaEngineGonkVideoSource.cpp
--- a/dom/media/webrtc/MediaEngineGonkVideoSource.cpp
+++ b/dom/media/webrtc/MediaEngineGonkVideoSource.cpp
@@ -77,17 +77,17 @@ MediaEngineGonkVideoSource::NotifyPull(M
 
   MonitorAutoLock lock(mMonitor);
   // B2G does AddTrack, but holds kStarted until the hardware changes state.
   // So mState could be kReleased here. We really don't care about the state,
   // though.
 
   // Note: we're not giving up mImage here
   nsRefPtr<layers::Image> image = mImage;
-  StreamTime delta = aDesiredTime - aLastEndTime;
+  StreamTime delta = aDesiredTime - aSource->GetEndOfAppendedData(aID);
   LOGFRAME(("NotifyPull, desired = %ld, delta = %ld %s", (int64_t) aDesiredTime,
             (int64_t) delta, image ? "" : "<null>"));
 
   // Bug 846188 We may want to limit incoming frames to the requested frame rate
   // mFps - if you want 30FPS, and the camera gives you 60FPS, this could
   // cause issues.
   // We may want to signal if the actual frame rate is below mMinFPS -
   // cameras often don't return the requested frame rate especially in low