Ignore seek operations to the current time in pause 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 b1744bd1c3d4893f8ed38aa8933ab804430bea90..600264168e5f245d2d735aa09badaaa888a6ae41 100644
--- a/media/blink/webmediaplayer_impl.cc
+++ b/media/blink/webmediaplayer_impl.cc
@@ -337,8 +337,20 @@ void WebMediaPlayerImpl::seek(double seconds) {
   media_log_->AddEvent(media_log_->CreateSeekEvent(seconds));
 
   // Update our paused time.
-  if (paused_)
-    paused_time_ = seek_time;
+  // In paused state ignore the seek operations to current time and generate
+  // buffer state change event to eventually fire seeking and seeked events
+  if (paused_) {
+    if (paused_time_ != seek_time) {
+      paused_time_ = seek_time;
+    }
+    else {

[DaleCurtis, 2014/10/29 18:31:46]

move else to upper line.

[Srirama, 2014/10/30 03:25:48]

Acknowledged.

+      main_task_runner_->PostTask(FROM_HERE,

[DaleCurtis, 2014/10/29 18:31:46]

Formatting is weird, have you run "git cl format" or used clang-format?

[Srirama, 2014/10/30 03:25:48]

Sorry, i will take care now.

+            base::Bind(&WebMediaPlayerImpl::OnPipelineBufferingStateChanged,
+            AsWeakPtr(),
+            BUFFERING_HAVE_ENOUGH));

[DaleCurtis, 2014/10/29 18:31:46]

What about just using whatever the current buffering state is? I.e. Directly
call SetReadyState(network_state_);

This also precludes a time changed event from happening, is the HTMLMediaElement
okay with that?

[philipj_slow, 2014/10/29 19:14:48]

Hmm, better have a test for that. There should be seeking, timeupdate and seeked
events, in that order.

[Srirama, 2014/10/30 03:25:48]

I will try with this option and will add a test

+      return;
+    }
+  }
 
   seeking_ = true;