Use loading period event time for fatal load errors.

ExoPlaybackExceptions of type SOURCE are always associated with the loading
period and thus we can use the event time for the loading period in
onPlayerError. Renderer and unexpected exceptions are still associated with the
currently playing period.

Issue:#5407
PiperOrigin-RevId: 230216253

RELEASENOTES.md

@@ -43,6 +43,9 @@
   fit service requirements. Remove ability to set a custom stop action.
   fit service requirements. Remove ability to set a custom stop action.
 * Add workaround for video quality problems with Amlogic decoders
 * Add workaround for video quality problems with Amlogic decoders
   ([#5003](https://github.com/google/ExoPlayer/issues/5003)).
   ([#5003](https://github.com/google/ExoPlayer/issues/5003)).
+* Associate fatal player errors of type SOURCE with the loading source in
+  `AnalyticsListener.EventTime`
+  ([#5407](https://github.com/google/ExoPlayer/issues/5407)).
 
 
 ### 2.9.4 ###
 ### 2.9.4 ###
 
 

library/core/src/main/java/com/google/android/exoplayer2/analytics/AnalyticsCollector.java

@@ -488,7 +488,10 @@ public class AnalyticsCollector
 
 
   @Override
   @Override
   public final void onPlayerError(ExoPlaybackException error) {
   public final void onPlayerError(ExoPlaybackException error) {
-    EventTime eventTime = generatePlayingMediaPeriodEventTime();
+    EventTime eventTime =
+        error.type == ExoPlaybackException.TYPE_SOURCE
+            ? generateLoadingMediaPeriodEventTime()
+            : generatePlayingMediaPeriodEventTime();
     for (AnalyticsListener listener : listeners) {
     for (AnalyticsListener listener : listeners) {
       listener.onPlayerError(eventTime, error);
       listener.onPlayerError(eventTime, error);
     }
     }