Comment #12 on issue 20127 by [email protected]: Stalled event always
fires
http://code.google.com/p/chromium/issues/detail?id=20127
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=29235
------------------------------------------------------------------------
r29235 | [email protected] | 2009-10-15 18:29:50 -0700 (Thu, 15 Oct 2009)
| 18 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/filter_host.h?r1=29235&r2=29234
M
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/mock_filter_host.h?r1=29235&r2=29234
M
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline_impl.cc?r1=29235&r2=29234
M
http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline_impl.h?r1=29235&r2=29234
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/media/buffered_data_source.cc?r1=29235&r2=29234
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/media/buffered_data_source.h?r1=29235&r2=29234
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/media/buffered_data_source_unittest.cc?r1=29235&r2=29234
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webmediaplayer_impl.cc?r1=29235&r2=29234
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webmediaplayer_impl.h?r1=29235&r2=29234
Report stalled event correctly for <video>
BUG=20127
BUG=13568
TEST=Opens a web page, stalled event should be fire only if network
is actuall stalled.
What this patch does:
1. Report Loading / Idle according to whether we are actively
receiving data in the BuffereResourceLoader. This is done by
signaling the network events to BufferedDataSource and then
to the media playback pipeline.
2. Report byteLoaded() as the last byte position buffered. This
will enable an actual ticking progress for the progress event.
With this value actually ticking, stalled event is suppressed.
Review URL: http://codereview.chromium.org/269002
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
