[Bug 21364] Simplify Coded Frame Processing (remove overlapped frame)

https://www.w3.org/Bugs/Public/show_bug.cgi?id=21364

Aaron Colwell <acolwell@chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |acolwell@chromium.org
           Assignee|adrianba@microsoft.com      |acolwell@chromium.org

--- Comment #1 from Aaron Colwell <acolwell@chromium.org> ---
(In reply to comment #0)
> The Coded Frame Processing says:
> 
> 13. If last decode timestamp for track buffer is unset and there is a coded
> frame in track buffer with a presentation timestamp less than or equal to
> presentation timestamp and presentation timestamp is less than this coded
> frame's presentation timestamp plus its frame duration, then run the
> following steps:
> 1. Let overlapped frame be the coded frame in track buffer that matches the
> condition above.
> 2. If track buffer contains audio coded frames, then run the audio splice
> frame algorithm and if a splice frame is returned, assign it to spliced
> frame.
> 3. Let overlapped frame presentation timestamp equal the presentation
> timestamp of overlapped frame.
> 4. Let remove window timestamp equal overlapped frame presentation timestamp
> plus 1 microsecond.
> 5. If track buffer contains video coded frames and the presentation
> timestamp is greater than or equal to the overlapped frame presentation
> timestamp and the presentation timestamp is less than the remove window
> timestamp, then remove overlapped frame and any coded frames that depend on
> it from track buffer.
> 
> Step 5 is unnecessarily complicated: According to 1, the overlapped frame
> satisfies the conditions in 13 so a) the track buffer contains video coded
> frames 

Nothing in 13 singles out video coded frames specifically so the 'contains
video coded frames' needs to stay.

and b) the presentation timestamp is greater or equal to the
> overlapped frame presentation timestamp.

I agree this one can be dropped.

> 
> Step 5 can be rewritten as:
> 
> 5. If the presentation timestamp is less than the remove window timestamp,
> then remove overlapped frame and any coded frames that depend on it from
> track buffer.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.

Received on Monday, 25 March 2013 20:54:34 UTC