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

Unified Diff: content/renderer/media/android/webmediaplayer_android.cc

Issue 2440563004: Switch to using an explicit ended signal instead of time comparison. (Closed)
Patch Set: Fix tests. Created 4 years, 2 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
Index: content/renderer/media/android/webmediaplayer_android.cc
diff --git a/content/renderer/media/android/webmediaplayer_android.cc b/content/renderer/media/android/webmediaplayer_android.cc
index 14c38d01cd05223037552f48b314f69b6b375922..dfc8c2bb7c32f1e315551e88897fff222c03a09e 100644
--- a/content/renderer/media/android/webmediaplayer_android.cc
+++ b/content/renderer/media/android/webmediaplayer_android.cc
@@ -716,7 +716,7 @@ void WebMediaPlayerAndroid::OnPlaybackComplete() {
// know that the playback has finished. To solve this, we set the
// current time to media duration when OnPlaybackComplete() get called.
interpolator_.SetBounds(duration_, duration_, default_tick_clock_.NowTicks());
- client_->timeChanged();
+ client_->timeChanged(true);
// If the loop attribute is set, timeChanged() will update the current time
// to 0. It will perform a seek to 0. Issue a command to the player to start
@@ -757,7 +757,7 @@ void WebMediaPlayerAndroid::OnSeekComplete(
UpdateReadyState(WebMediaPlayer::ReadyStateHaveEnoughData);
- client_->timeChanged();
+ client_->timeChanged(playback_completed_);
}
void WebMediaPlayerAndroid::OnMediaError(int error_type) {
« no previous file with comments | « no previous file | content/renderer/media/webmediaplayer_ms_unittest.cc » ('j') | media/filters/ffmpeg_demuxer.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698