Don't suspend the pipeline before HaveFutureData while decoding progressing

media/blink/webmediaplayer_impl.h

Index: media/blink/webmediaplayer_impl.h
diff --git a/media/blink/webmediaplayer_impl.h b/media/blink/webmediaplayer_impl.h
index 729d41ec4334c2d0b0c9ab65c59daad4e9fb77ef..8b3b013e45fb69730e4d94cdba917451f4e6f08e 100644
--- a/media/blink/webmediaplayer_impl.h
+++ b/media/blink/webmediaplayer_impl.h
@@ -18,6 +18,7 @@
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "base/threading/thread.h"
+#include "base/time/default_tick_clock.h"
 #include "base/timer/elapsed_timer.h"
 #include "base/timer/timer.h"
 #include "build/build_config.h"
@@ -563,6 +564,13 @@ class MEDIA_BLINK_EXPORT WebMediaPlayerImpl
   int underflow_count_;
   std::unique_ptr<base::ElapsedTimer> underflow_timer_;
 
+  // The last time didLoadingProgress() returned true.
+  base::TimeTicks last_time_loading_progressed_;
+
+  // Points to |default_tick_clock_| by default, but can be overridden by tests.
+  base::TickClock* tick_clock_;

[DaleCurtis, 2016/10/28 19:11:42]

Nowadays I like just having a single std::unique_ptr<base::TickClock> member
that a test can set.

+  base::DefaultTickClock default_tick_clock_;
+
   // Monitors the player events.
   base::WeakPtr<MediaObserver> observer_;