Comment #15 on issue 16056 by [email protected]: [HTML5-Video] layout  
test on buffered attribute fails in Chrome
http://code.google.com/p/chromium/issues/detail?id=16056

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=22822

------------------------------------------------------------------------
r22822 | [email protected] | 2009-08-07 16:08:23 -0700 (Fri, 07 Aug  
2009) | 26 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/branches/195/src/media/base/mock_filters.h?r1=22822&r2=22821
    M  
http://src.chromium.org/viewvc/chrome/branches/195/src/media/base/pipeline_impl.cc?r1=22822&r2=22821
    M  
http://src.chromium.org/viewvc/chrome/branches/195/src/media/base/pipeline_impl_unittest.cc?r1=22822&r2=22821

Merge 22087 - Missing buffered attribute for <video>/<audio>

BUG=16056
TEST=LayoutTests/media/videobuffered.html

The current implementation of the buffered attribute for <video> and <audio>
is broken. There are several problems around this attribute:

1. We don't have any caching on disk, so we only keep a recent range of  
bytes
    close to current playback position.
2. WebKit reports buffered as one range (0, max_time_buffered). But we only
    cache a short partial range which doesn't start with 0. The correct
    implementation is a list of ranges buffered. But this has to go into  
WebKit
    first.
3. We don't have an accurate mapping between byte offset <  > timestamp.

So the current implementation is to lie about what we have buffered. We  
always
say we have buffered everything before the current download position. And we
only report one range. The calculation of time is also based on scaling the
duration with current buffered bytes.

Review URL: http://codereview.chromium.org/160300

[email protected]

Review URL: http://codereview.chromium.org/165180
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to