diff options
| author | Monroe Linden <monroe@lindenlab.com> | 2010-06-15 10:53:02 -0700 |
|---|---|---|
| committer | Monroe Linden <monroe@lindenlab.com> | 2010-06-15 10:53:02 -0700 |
| commit | d6af2d0985951f5863620bfc6d6e763769cc07fa (patch) | |
| tree | 5da98c77afc0e36910a23f860fefc0d07c1e4019 /indra/newview/skins/default/xui/fr/menu_url_agent.xml | |
| parent | da078d9a35e256a59652f6ed545b3cb09f384bb0 (diff) | |
Fix for EXT-7414 (Jumping to a specific time in a video stream does not work in Viewer 2.)
This was a race condition. In the repro case, LLViewerMediaImpl::seek() was getting called before the media source got a chance to be loaded (since the load is now delayed at least 1 frame due to the priority calculations), so the seek command was being dropped.
The fix is to save the seek time in mPreviousMediaTime in this case, and rearrange the code in LLViewerMediaImpl::loadURI() so that it gets used whenever it's non-zero. I've also added similar code in LLViewerMediaImpl::pause() and LLViewerMediaImpl::start() to set mPreviousMediaState. (If there are any cases where content wants parcel media to start out paused, they would have had a similar issue.)
Reviewed by Callum and Richard at http://codereview.lindenlab.com/2478002 .
Diffstat (limited to 'indra/newview/skins/default/xui/fr/menu_url_agent.xml')
0 files changed, 0 insertions, 0 deletions