Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(337)

Unified Diff: content/browser/media/android/media_player_renderer.cc

Issue 2626443003: Improve MediaPlayerBridge seek resiliance (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | media/base/android/media_player_bridge.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/media/android/media_player_renderer.cc
diff --git a/content/browser/media/android/media_player_renderer.cc b/content/browser/media/android/media_player_renderer.cc
index b2e6f52de1cf2855abbbffe3b228ff33b0ee18e5..f09f21ce252cb711ac8c18b8cb63250d9b7890ac 100644
--- a/content/browser/media/android/media_player_renderer.cc
+++ b/content/browser/media/android/media_player_renderer.cc
@@ -109,13 +109,9 @@ void MediaPlayerRenderer::StartPlayingFrom(base::TimeDelta time) {
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
« no previous file with comments | « no previous file | media/base/android/media_player_bridge.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698