Relanding 'Ignore seek operations to the current time in pause state' patch

media/base/pipeline_unittest.cc

Index: media/base/pipeline_unittest.cc
diff --git a/media/base/pipeline_unittest.cc b/media/base/pipeline_unittest.cc
index 9528cf7b3b4bb40e800ffcd4ff2eb5d6eed6569b..ddc73095cb61edf29a89e8e57215d6d28dfd2b00 100644
--- a/media/base/pipeline_unittest.cc
+++ b/media/base/pipeline_unittest.cc
@@ -619,6 +619,9 @@ TEST_F(PipelineTest, EndedCallback) {
   message_loop_.RunUntilIdle();
 
   EXPECT_CALL(callbacks_, OnEnded());
+  // There are cases where duration is reported wrong initially, so there is

[DaleCurtis, 2014/12/02 18:14:05]

Hmm, doesn't this add a  bunch of log spam / errors in some cases where it
doesn't get triggered?

[Srirama, 2014/12/03 14:43:02]

I thought about it initially but after seeing the local unittest results and bot
results i thought it is ok. I don't know if i can do it conditionally similar to
what i am doing in RunEndedCallbackIfNeeded(). What do you suggest?

[DaleCurtis, 2014/12/04 02:04:41]

If there's just a few tests which trigger this OnDuration change you should
explicitly add it to those tests. If there's a bunch (> 5) you can consider
switching to a NiceMock; though we'll want to hunt down what the real issue is
here.

[Srirama, 2014/12/04 03:16:46]

will check it.

+  // an onDurationChange event fired again on OnEnded event if required.

[DaleCurtis, 2014/12/02 18:14:05]

s/onDuration/OnDuration/

[Srirama, 2014/12/03 14:43:02]

Done.

+  EXPECT_CALL(callbacks_, OnDurationChange());
   text_stream()->SendEosNotification();
   message_loop_.RunUntilIdle();
 }