Last Comment Bug 761708 - Wrong |played| value when seeking back in the media.
: Wrong |played| value when seeking back in the media.
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Audio/Video (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla16
Assigned To: Paul Adenot (:padenot)
:
:
Mentors:
Depends on: 768583
Blocks:
  Show dependency tree / graph
 
Reported: 2012-06-05 10:56 PDT by Paul Adenot (:padenot)
Modified: 2012-06-26 11:43 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Attachments
Patch v0 : fix the bug and add a test. (1.87 KB, patch)
2012-06-05 10:58 PDT, Paul Adenot (:padenot) 		cajbir.bugzilla: review+
Details | Diff | Splinter Review
Adding r=chris.double (2.05 KB, patch)
2012-06-05 18:42 PDT, Paul Adenot (:padenot) 		no flags Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Description Paul Adenot (:padenot) 2012-06-05 10:56:52 PDT 
STR:
- Load a video
- Play for some time
- Seek back in the stream

Expected:
- |played| should be [0.00, time we seeked from].

Actual result:
- |played| is [0.00, time we seeked to].

This is caused by the |nsTimeRanges::Normalize| method, which has a bug, it does not handle ranges contained in other ranges.
Comment 1 Paul Adenot (:padenot) 2012-06-05 10:58:24 PDT 
Created attachment 630227 [details] [diff] [review]
Patch v0 : fix the bug and add a test.
Comment 2 Paul Adenot (:padenot) 2012-06-05 18:42:38 PDT 
Created attachment 630403 [details] [diff] [review]
Adding r=chris.double
Comment 4 Ed Morley [:emorley] 2012-06-07 05:47:31 PDT 
https://hg.mozilla.org/mozilla-central/rev/849da4232313
Note You need to log in before you can comment on or make changes to this bug.