Video plays on server without range request

BUG=19521
TEST=Go to http://htmlfive.appspot.com/static/video.html and it will play.

r23255 introduced a regression that causes an extra seek in the
beginning of playing a video file. This extra seek causes FFmpeg
to look for index at the end of file which issue a range request
at the end of the file.

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@23598 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 7 insertions, 3 deletions

diff --git a/webkit/glue/webmediaplayer_impl.cc b/webkit/glue/webmediaplayer_impl.cc
index 5df50d4..00c6834 100644
--- a/webkit/glue/webmediaplayer_impl.cc
+++ b/webkit/glue/webmediaplayer_impl.cc
@@ -262,9 +262,13 @@ bool WebMediaPlayerImpl::supportsSave() const {
 void WebMediaPlayerImpl::seek(float seconds) {
   DCHECK(MessageLoop::current() == main_loop_);
 
-  // TODO(scherkus): WebKit fires a seek(0) at the very start, however pipeline
-  // already does a seek(0) internally.  Investigate whether doing two seek(0)
-  // at the start impacts startup latency.
+  // WebKit fires a seek(0) at the very start, however pipeline already does a
+  // seek(0) internally.  Avoid doing seek(0) the second time because this will
+  // cause extra pre-rolling and will break servers without range request
+  // support.
+  if (pipeline_->GetCurrentTime().ToInternalValue() == 0 && seconds == 0) {
+    return;
+  }
 
   // Try to preserve as much accuracy as possible.
   float microseconds = seconds * base::Time::kMicrosecondsPerSecond;