Prevent seeking when Android MediaPlayer reports that it's not possible.

media/base/android/media_player_bridge.h

Index: media/base/android/media_player_bridge.h
diff --git a/media/base/android/media_player_bridge.h b/media/base/android/media_player_bridge.h
index 291aa97959dbfa7dd4136cd6db5d7422bbbdeea7..b63a49a6d6e61553b59a3e78e6ed3696cbb50c86 100644
--- a/media/base/android/media_player_bridge.h
+++ b/media/base/android/media_player_bridge.h
@@ -99,13 +99,15 @@ class MEDIA_EXPORT MediaPlayerBridge : public MediaPlayerAndroid {
   virtual base::android::ScopedJavaLocalRef<jobject> GetAllowedOperations();
 
  private:
+  friend class MediaPlayerBridgeTest;
+
   // Set the data source for the media player.
   void SetDataSource(const std::string& url);
 
   // Functions that implements media player control.
   void StartInternal();
   void PauseInternal();
-  void SeekInternal(base::TimeDelta time);
+  bool SeekInternal(base::TimeDelta current_time, base::TimeDelta time);

[qinmin, 2015/07/17 04:16:28]

the current_time can always be retrieved from getCurrentTime(), why not just
passing the seek_time in here.

Please also document the return value

[keitchen, 2015/07/20 20:24:23]

GetCurrentTime() issues a JNI call to the media player to retrieve the current
time value. I wanted the ability to set this value when unit testing
SeekInternal().

Added documentation for the return value.

 
   // Called when |time_update_timer_| fires.
   void OnTimeUpdateTimerFired();