From 9fa2aaf33d9060e9950f3b87e4a41900e9bca5f7 Mon Sep 17 00:00:00 2001 From: tonihei Date: Mon, 11 May 2020 14:05:41 +0100 Subject: [PATCH] Use event playback position in EventLogger. We currently use the playback position in the current window, not the position in the window the event belongs to. This creates confusing outputs, e.g, window=1, mediaPos=100, even if the position refers to window 0. issue:#7332 PiperOrigin-RevId: 310896908 --- .../java/com/google/android/exoplayer2/util/EventLogger.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/library/core/src/main/java/com/google/android/exoplayer2/util/EventLogger.java b/library/core/src/main/java/com/google/android/exoplayer2/util/EventLogger.java index 3136556f2c..04e10472c9 100644 --- a/library/core/src/main/java/com/google/android/exoplayer2/util/EventLogger.java +++ b/library/core/src/main/java/com/google/android/exoplayer2/util/EventLogger.java @@ -553,7 +553,7 @@ public class EventLogger implements AnalyticsListener { return "eventTime=" + getTimeString(eventTime.realtimeMs - startTimeMs) + ", mediaPos=" - + getTimeString(eventTime.currentPlaybackPositionMs) + + getTimeString(eventTime.eventPlaybackPositionMs) + ", " + windowPeriodString; }