Pass seek event through pipeline during loading state

media/blink/webmediaplayer_impl.cc

Index: media/blink/webmediaplayer_impl.cc
diff --git a/media/blink/webmediaplayer_impl.cc b/media/blink/webmediaplayer_impl.cc
index 9f0c53db268226243b0244ab537c6ec8d81fe47f..4e1e49f32364bfd0c9a3de197b99a64760771b4d 100644
--- a/media/blink/webmediaplayer_impl.cc
+++ b/media/blink/webmediaplayer_impl.cc
@@ -310,6 +310,7 @@ void WebMediaPlayerImpl::seek(double seconds) {
 
   ended_ = false;
 
+  ReadyState old_state = ready_state_;
   if (ready_state_ > WebMediaPlayer::ReadyStateHaveMetadata)
     SetReadyState(WebMediaPlayer::ReadyStateHaveMetadata);
 
@@ -330,7 +331,7 @@ void WebMediaPlayerImpl::seek(double seconds) {
   // OnPipelineSeeked and OnPipelineBufferingStateChanged events
   // to eventually fire seeking and seeked events
   if (paused_) {
-    if (paused_time_ != seek_time) {
+    if (paused_time_ != seek_time || old_state != ReadyStateHaveEnoughData) {

[DaleCurtis, 2015/01/06 19:18:55]

Can you see if this allows the removal of OnPipelineSeeked() now? I.e. move the
inverse of this state check to the PostTask for
OnPipelineBufferingStateChanged(). 

if (paused_time_ != seek_time) {
  paused_time_ = seek_time;
} else if (old_state == ReadyStateHaveEnoughData) {
  // PostTask(OnPipelineBufferingStateChanged);
  return;
}

This will avoid redoing the seek to the beginning if load is still in progress. 
I also don't remember exactly why the OnPipelineSeeked() is necessary. The load
event should eventually completely and issue the correct buffering state change.

       paused_time_ = seek_time;
     } else {
       main_task_runner_->PostTask(