Improve MediaPlayerBridge seek resiliance

content/browser/media/android/media_player_renderer.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/media/android/media_player_renderer.h"
 
 #include <memory>
 
 #include "base/callback_helpers.h"
 #include "content/browser/android/scoped_surface_request_manager.h"
@@ skipping 91 matching lines @@
   DVLOG(3) << __func__;
   flush_cb.Run();
 }
 
 void MediaPlayerRenderer::StartPlayingFrom(base::TimeDelta time) {
   // MediaPlayerBridge's Start() is idempotent, except when it has encountered
   // an error (in which case, calling Start() again is logged as a new error).
   if (has_error_)
     return;
 
+  media_player_->SeekTo(time);
   media_player_->Start();
 
-  // Drop the seek if we are live streaming or if we don't have any duration
-  // information yet.
-  if (duration_ != media::kInfiniteDuration)
-    media_player_->SeekTo(time);
-
   // WMPI needs to receive a BUFFERING_HAVE_ENOUGH data before sending a
   // playback_rate > 0. The MediaPlayer manages its own buffering and will pause
   // internally if ever it runs out of data. Sending BUFFERING_HAVE_ENOUGH here
   // is always safe.
   //
   // NOTE: OnBufferingUpdate is triggered whenever the media has buffered or
   // played up to a % value between 1-100, and it's not a reliable indicator of
   // the buffering state.
   //
   // TODO(tguilbert): Investigate the effect of this call on UMAs.
@@ skipping 148 matching lines @@
 void MediaPlayerRenderer::CancelScopedSurfaceRequest() {
   if (!surface_request_token_)
     return;
 
   ScopedSurfaceRequestManager::GetInstance()->UnregisterScopedSurfaceRequest(
       surface_request_token_);
   surface_request_token_ = base::UnguessableToken();
 }
 
 }  // namespace content