New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 131444 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Jun 2012
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Restricted
  • Only users with EditIssue permission may comment.



Sign in to add a comment

Buffered area is missing after I seek to unbuffered area

Project Member Reported by alek...@chromium.org, Jun 6 2012

Issue description

Chrome Version       : 21.0.1165.0 
URLs (if applicable) : http://tskir-html5.kir.corp.google.com/testmatrix/index.html?file=sync2.webm
OS version               : Mac 
Flash or HTML5?  <right-clicking most players will either reveal some text with “Flash”; otherwise likely HTML5> HTML5 

What steps will reproduce the problem?
1. go to this url http://tskir-html5.kir.corp.google.com/testmatrix/index.html?file=sync2.webm 
2. select Cable then Wi-Fi 

What is the expected result?
buffering blue line should be gradually populated the seek bar 

What is the actual result?
buffering blue line disappears 
 

Comment 1 Deleted

@fischman, can you take a look please?
it might related to this change in CL: https://code.google.com/p/chromium/issues/detail?id=131444 
Note: sometimes you need to click Cable and then Wi-Fi buttons fast to reproduce the issue 
see screenshot 
Screen Shot 2012-06-06 at 2.02.20 PM.png
55.8 KB View Download

Comment 4 by sh...@chromium.org, Jun 6 2012

Kenji and I were able to repro on Windows 7 as well.
Labels: OS-All
This sounds like a dup of 131225 except you say you see this in 21.0.1165.0 which http://omahaproxy.appspot.com/revision?version=21.0.1165.0 tells me was cut from r140711 which is later than the fix for 131225 in r140536.

Can you repro this in a debug build?  Does it really not change if you revert/reapply r140536?
Status: Untriaged
Happens on Version 21.0.1171.0, and now it doesn't have to be on wifi, I just move seek bar to unbuffered area and blue seek bar is gone 

Summary: Buffering blue line is gone when moving seek bar to unbuffered area
alekyoo: as in #6: can you repro this in a debug build?  Does it really not change if you revert/reapply r140536?

Comment 11 by alekyoo@google.com, Jun 11 2012

fischman@ sorry I don't build chrome myself
Labels: ReleaseBlock-Stable
Labels: Mstone-21
Owner: fischman@chromium.org
Status: Assigned
Status: WontFix
I can repro the blue bar disappearing following a seek, but it then returns once enough data is buffered following the new current time (post seek).  This is working-as-intended; the point of  bug 103513  (and the several that cascaded from it) is that we should not claim to have buffered regions of the media that were skipped over (because of seeking).  Instead, we now only show the buffered region containing the current media time.  Following a seek, especially on a slow link, it takes time to buffer enough of the new area, so the blue bar "disappears" until we have enough buffered to show usefully.

If you see the blue bar disappear and never come back though it has some time to, please reopen the bug.  Closing as WaI until then.
(if that's the case, please also let me know if you can repro it on linux; I cannot repro the bar staying gone on linux)
Status: Assigned
Summary: Buffered area is missing after I seek to unbuffered area
I am attaching the screenshot 21.0.1174.0 canary on Windows. 

What I did is as follows:
1) http://tskir-html5.kir.corp.google.com/testmatrix/mediaFiles/3dtv/3dtv1.webm
2) seek to unbuffered area
3) the buffer area dissapers
4) it will keep on playing but buffered area never comes back.

I will look into Unix now. However, I am reopening this and changing title.

Buffer.png
20.9 KB View Download
same happens on Version 21.0.1174.0 canary on Mac 

Screen Shot 2012-06-15 at 8.58.38 AM.png
58.8 KB View Download
@imasaki: the screenshot you attached *does* show the buffered area returning - note the bolder white around the thumb.  Highlighted in https://screenshot.googleplex.com/RXGugRbQZ (google-internal-only).
Status: WontFix
Discussed with  fischman@ offline. This behavior is "Work as intended". Closing this bug.

I am sorry for miscommunication.

Project Member

Comment 19 by bugdroid1@chromium.org, Jun 23 2012

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

------------------------------------------------------------------------
r143765 | fischman@chromium.org | Fri Jun 22 18:04:53 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/tools/seek_tester/seek_tester.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/mock_data_source_host.h?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/filters/dummy_demuxer.h?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/data_source.h?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/ranges.h?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/filters/ffmpeg_demuxer_unittest.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/filters/chunk_demuxer_unittest.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/filters/ffmpeg_demuxer.h?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/media.gyp?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/filters/ffmpeg_demuxer.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/filters/chunk_demuxer.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline_unittest.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/pipeline.h?r1=143765&r2=143764&pathrev=143765
 A http://src.chromium.org/viewvc/chrome/trunk/src/media/base/ranges.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/demuxer_stream.h?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/mock_demuxer_host.h?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/filters/dummy_demuxer.cc?r1=143765&r2=143764&pathrev=143765
 M http://src.chromium.org/viewvc/chrome/trunk/src/media/base/mock_filters.h?r1=143765&r2=143764&pathrev=143765

Ensure media's buffered ranges always have a range that includes currentTime.
Avoids buffering bar disappearing/reappearing when the bytes are distributed
unevenly throughout the media.

BUG= 133567 , 131444 

Review URL: https://chromiumcodereview.appspot.com/10581050
------------------------------------------------------------------------
Project Member

Comment 20 by bugdroid1@chromium.org, Jul 10 2012

Labels: merge-merged-1180
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=145937

------------------------------------------------------------------------
r145937 | fischman@chromium.org | Tue Jul 10 12:55:34 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/ranges.h?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_filters.h?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/tools/seek_tester/seek_tester.cc?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_data_source_host.h?r1=145937&r2=145936&pathrev=145937
 A http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/ranges.cc?r1=145937&r2=145936&pathrev=145937 (from /trunk/src/media/base/ranges.cc revision 143765)
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/media.gyp?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_demuxer_host.h?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/data_source.h?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline.cc?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline_unittest.cc?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/filters/ffmpeg_demuxer_unittest.cc?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline.h?r1=145937&r2=145936&pathrev=145937
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/demuxer_stream.h?r1=145937&r2=145936&pathrev=145937

Merge 143765 - Ensure media's buffered ranges always have a range that includes currentTime.
Avoids buffering bar disappearing/reappearing when the bytes are distributed
unevenly throughout the media.

BUG= 133567 , 131444 

Review URL: https://chromiumcodereview.appspot.com/10581050

TBR=fischman@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10698134
------------------------------------------------------------------------
Project Member

Comment 21 by bugdroid1@chromium.org, Jul 10 2012

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

------------------------------------------------------------------------
r145951 | fischman@chromium.org | Tue Jul 10 13:27:58 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/ranges.h?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_filters.h?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/tools/seek_tester/seek_tester.cc?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_data_source_host.h?r1=145951&r2=145950&pathrev=145951
 D http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/ranges.cc?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/media.gyp?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_demuxer_host.h?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/data_source.h?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline.cc?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline_unittest.cc?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/filters/ffmpeg_demuxer_unittest.cc?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline.h?r1=145951&r2=145950&pathrev=145951
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/demuxer_stream.h?r1=145951&r2=145950&pathrev=145951

Revert 145937 - Merge 143765 - Ensure media's buffered ranges always have a range that includes currentTime.
Avoids buffering bar disappearing/reappearing when the bytes are distributed
unevenly throughout the media.

BUG= 133567 , 131444 

Review URL: https://chromiumcodereview.appspot.com/10581050

TBR=fischman@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10698134

TBR=fischman@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10689141
------------------------------------------------------------------------
Project Member

Comment 22 by bugdroid1@chromium.org, Jul 10 2012

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

------------------------------------------------------------------------
r145966 | fischman@chromium.org | Tue Jul 10 15:12:23 PDT 2012

Changed paths:
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_filters.h?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/tools/seek_tester/seek_tester.cc?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_data_source_host.h?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/data_source.h?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline.cc?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/filters/ffmpeg_demuxer_unittest.cc?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/filters/ffmpeg_demuxer.h?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/filters/dummy_demuxer.h?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/ranges.h?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/filters/chunk_demuxer.cc?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/media.gyp?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/mock_demuxer_host.h?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/filters/dummy_demuxer.cc?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/filters/ffmpeg_demuxer.cc?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline_unittest.cc?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/pipeline.h?r1=145966&r2=145965&pathrev=145966
 M http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/demuxer_stream.h?r1=145966&r2=145965&pathrev=145966

Merge 143765 - Ensure media's buffered ranges always have a range that includes currentTime.
Avoids buffering bar disappearing/reappearing when the bytes are distributed
unevenly throughout the media.

THIS IS A MANUAL (not drover'd) MERGE.
Due to preceding *unmerged* changes (esp r143106) to
chunk_demuxer{,_unittest}.cc, the original CL doesn't apply cleanly to the 1180
branch.  Instead these two files are omitted from the merge, and only enough
bits are changed in chunk_demuxer.cc to allow compilation to proceed.  Also,
ranges.h needed IntersectionWith from the above unmerged CL added manually.

BUG= 133567 , 131444 
TBR=fischman@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10704142
------------------------------------------------------------------------
Project Member

Comment 23 by bugdroid1@chromium.org, Jul 10 2012

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

------------------------------------------------------------------------
r145974 | fischman@chromium.org | Tue Jul 10 15:49:28 PDT 2012

Changed paths:
 A http://src.chromium.org/viewvc/chrome/branches/1180/src/media/base/ranges.cc?r1=145974&r2=145973&pathrev=145974

Add missing file from r145966.

BUG= 133567 , 131444 
TBR=fischman@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10689147
------------------------------------------------------------------------
Project Member

Comment 24 by bugdroid1@chromium.org, Oct 13 2012

Labels: Restrict-AddIssueComment-Commit
This issue has been closed for some time. No one will pay attention to new comments.
If you are seeing this bug or have new data, please click New Issue to start a new bug.
Project Member

Comment 25 by bugdroid1@chromium.org, Mar 10 2013

Labels: -Feature-Media -Area-WebKit -Mstone-21 Cr-Content Cr-Internals-Media M-21
Project Member

Comment 26 by bugdroid1@chromium.org, Mar 14 2013

Labels: -Restrict-AddIssueComment-Commit Restrict-AddIssueComment-EditIssue
Project Member

Comment 27 by bugdroid1@chromium.org, Apr 6 2013

Labels: -Cr-Content Cr-Blink

Sign in to add a comment