Enable seek in fullscreen mode for MSE impl on android

webkit/media/android/webmediaplayer_android.cc

Index: webkit/media/android/webmediaplayer_android.cc
diff --git a/webkit/media/android/webmediaplayer_android.cc b/webkit/media/android/webmediaplayer_android.cc
index 973c5f29bbda113c9df61ffc16523e42cc356e09..4f64d183c284a81263dcd0893eb4208cc08f8a7e 100644
--- a/webkit/media/android/webmediaplayer_android.cc
+++ b/webkit/media/android/webmediaplayer_android.cc
@@ -158,8 +158,12 @@ void WebMediaPlayerAndroid::seek(double seconds) {
   seeking_ = true;
 
   base::TimeDelta seek_time = ConvertSecondsToTimestamp(seconds);
+#if defined(GOOGLE_TV)

[acolwell GONE FROM CHROMIUM, 2013/05/23 18:21:30]

At a minimum, this code MUST arrange for chunk_demuxer_->CancelPendingSeek() and
chunk_demuxer_->StartWaitingForSeek() inside this call stack like
WebMediaPlayerImpl does. If you don't it can cause playback to deadlock when
there isn't enough data to complete a seek. The call to chunk_demuxer_->Seek()
can be deferred to a later point in time though.

[qinmin, 2013/05/23 20:07:21]

Done. MediaSourceDelegate::Seek() already packs in
chunk_demuxer->StartWaitingForSeek(). Added a call to cancelPendingSeek(). 

On 2013/05/23 18:21:30, acolwell wrote:

+  // TODO(qinmin): check if GTV can also defer the seek until the browser side
+  // player is ready.
   if (media_source_delegate_)
     media_source_delegate_->Seek(seek_time);
+#endif
   if (proxy_)
     proxy_->Seek(player_id_, seek_time);
 }
@@ -499,6 +503,13 @@ void WebMediaPlayerAndroid::OnMediaPlayerPause() {
   client_->playbackStateChanged();
 }
 
+void WebMediaPlayerAndroid::OnMediaSeekRequest(base::TimeDelta time_to_seek) {
+  if (media_source_delegate_) {
+    media_source_delegate_->Seek(time_to_seek);
+    OnTimeUpdate(time_to_seek);
+  }
+}
+
 void WebMediaPlayerAndroid::UpdateNetworkState(
     WebMediaPlayer::NetworkState state) {
   network_state_ = state;