Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(192)

Issue 2192001: Reporting a more accurate buffered time for the video tag (Closed)

Created:
10 years, 7 months ago by Alpha Left Google
Modified:
9 years, 6 months ago
Reviewers:
CC:
chromium-reviews_googlegroups.com, fbarchard, Alpha Left Google, darin-cc_chromium.org, awong, Paweł Hajdan Jr., scherkus (not reviewing)
Visibility:
Public.

Description

Reporting a more accurate buffered time for the video tag Instead of showing that the entire file is buffered when the video first loads, this uses current time, current bytes, and buffered bytes to estimate buffered time. Reviewed: http://codereview.chromium.org/2085012/ Submitted for: vrk@chromium.org BUG=42285 TEST=NONE Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=48199

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+71 lines, -13 lines) Patch
M media/base/filter_host.h View 1 chunk +6 lines, -0 lines 0 comments Download
M media/base/mock_filter_host.h View 1 chunk +2 lines, -0 lines 0 comments Download
M media/base/pipeline.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/pipeline_impl.h View 3 chunks +12 lines, -1 line 0 comments Download
M media/base/pipeline_impl.cc View 3 chunks +29 lines, -10 lines 0 comments Download
M media/base/pipeline_impl_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/ffmpeg_demuxer.cc View 1 chunk +3 lines, -0 lines 0 comments Download
M media/filters/ffmpeg_demuxer_unittest.cc View 2 chunks +2 lines, -0 lines 0 comments Download
M webkit/glue/webmediaplayer_impl.h View 1 chunk +4 lines, -0 lines 0 comments Download
M webkit/glue/webmediaplayer_impl.cc View 1 chunk +11 lines, -0 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698