Some minor tweaks

1. Add EventLogger right away in PlayerActivity, else it doesn't log
   playWhenReady being initially set to true.
2. Remove EventLogger logging for the audio position advancing. It's
   redundant with isPlaying logging unless you're very specifically
   interested in the timing difference.
3. Remove unnecessary comment in Player.
4. Fix Timeline Javadoc.

PiperOrigin-RevId: 328983944

demos/main/src/main/java/com/google/android/exoplayer2/demo/PlayerActivity.java

@@ -297,9 +297,9 @@ public class PlayerActivity extends AppCompatActivity
               .setTrackSelector(trackSelector)
               .build();
       player.addListener(new PlayerEventListener());
+      player.addAnalyticsListener(new EventLogger(trackSelector));
       player.setAudioAttributes(AudioAttributes.DEFAULT, /* handleAudioFocus= */ true);
       player.setPlayWhenReady(startAutoPlay);
-      player.addAnalyticsListener(new EventLogger(trackSelector));
       playerView.setPlayer(player);
       playerView.setPlaybackPreparer(this);
       debugViewHelper = new DebugTextViewHelper(player, debugTextView);

library/core/src/main/java/com/google/android/exoplayer2/Player.java

@@ -374,8 +374,6 @@ public interface Player {
   }
 
   /** The device component of a {@link Player}. */
-  // Note: It's mostly from the androidx.media.VolumeProviderCompat and
-  //  androidx.media.MediaControllerCompat.PlaybackInfo.
   interface DeviceComponent {
 
     /** Adds a listener to receive device events. */

library/core/src/main/java/com/google/android/exoplayer2/Timeline.java

@@ -48,62 +48,74 @@ import com.google.android.exoplayer2.util.Util;
  * <h3 id="single-file">Single media file or on-demand stream</h3>
  *
  * <p style="align:center"><img src="doc-files/timeline-single-file.svg" alt="Example timeline for a
- * single file"> A timeline for a single media file or on-demand stream consists of a single period
+ * single file">
- * and window. The window spans the whole period, indicating that all parts of the media are
+ *
- * available for playback. The window's default position is typically at the start of the period
+ * <p>A timeline for a single media file or on-demand stream consists of a single period and window.
- * (indicated by the black dot in the figure above).
+ * The window spans the whole period, indicating that all parts of the media are available for
+ * playback. The window's default position is typically at the start of the period (indicated by the
+ * black dot in the figure above).
  *
  * <h3>Playlist of media files or on-demand streams</h3>
  *
  * <p style="align:center"><img src="doc-files/timeline-playlist.svg" alt="Example timeline for a
- * playlist of files"> A timeline for a playlist of media files or on-demand streams consists of
+ * playlist of files">
- * multiple periods, each with its own window. Each window spans the whole of the corresponding
+ *
- * period, and typically has a default position at the start of the period. The properties of the
+ * <p>A timeline for a playlist of media files or on-demand streams consists of multiple periods,
- * periods and windows (e.g. their durations and whether the window is seekable) will often only
+ * each with its own window. Each window spans the whole of the corresponding period, and typically
- * become known when the player starts buffering the corresponding file or stream.
+ * has a default position at the start of the period. The properties of the periods and windows
+ * (e.g. their durations and whether the window is seekable) will often only become known when the
+ * player starts buffering the corresponding file or stream.
  *
  * <h3 id="live-limited">Live stream with limited availability</h3>
  *
  * <p style="align:center"><img src="doc-files/timeline-live-limited.svg" alt="Example timeline for
- * a live stream with limited availability"> A timeline for a live stream consists of a period whose
+ * a live stream with limited availability">
- * duration is unknown, since it's continually extending as more content is broadcast. If content
+ *
- * only remains available for a limited period of time then the window may start at a non-zero
+ * <p>A timeline for a live stream consists of a period whose duration is unknown, since it's
- * position, defining the region of content that can still be played. The window will have {@link
+ * continually extending as more content is broadcast. If content only remains available for a
- * Window#isLive} set to true to indicate it's a live stream and {@link Window#isDynamic} set to
+ * limited period of time then the window may start at a non-zero position, defining the region of
- * true as long as we expect changes to the live window. Its default position is typically near to
+ * content that can still be played. The window will have {@link Window#isLive} set to true to
- * the live edge (indicated by the black dot in the figure above).
+ * indicate it's a live stream and {@link Window#isDynamic} set to true as long as we expect changes
+ * to the live window. Its default position is typically near to the live edge (indicated by the
+ * black dot in the figure above).
  *
  * <h3>Live stream with indefinite availability</h3>
  *
  * <p style="align:center"><img src="doc-files/timeline-live-indefinite.svg" alt="Example timeline
- * for a live stream with indefinite availability"> A timeline for a live stream with indefinite
+ * for a live stream with indefinite availability">
- * availability is similar to the <a href="#live-limited">Live stream with limited availability</a>
+ *
- * case, except that the window starts at the beginning of the period to indicate that all of the
+ * <p>A timeline for a live stream with indefinite availability is similar to the <a
- * previously broadcast content can still be played.
+ * href="#live-limited">Live stream with limited availability</a> case, except that the window
+ * starts at the beginning of the period to indicate that all of the previously broadcast content
+ * can still be played.
  *
  * <h3 id="live-multi-period">Live stream with multiple periods</h3>
  *
  * <p style="align:center"><img src="doc-files/timeline-live-multi-period.svg" alt="Example timeline
- * for a live stream with multiple periods"> This case arises when a live stream is explicitly
+ * for a live stream with multiple periods">
- * divided into separate periods, for example at content boundaries. This case is similar to the <a
+ *
- * href="#live-limited">Live stream with limited availability</a> case, except that the window may
+ * <p>This case arises when a live stream is explicitly divided into separate periods, for example
- * span more than one period. Multiple periods are also possible in the indefinite availability
+ * at content boundaries. This case is similar to the <a href="#live-limited">Live stream with
- * case.
+ * limited availability</a> case, except that the window may span more than one period. Multiple
+ * periods are also possible in the indefinite availability case.
  *
  * <h3>On-demand stream followed by live stream</h3>
  *
  * <p style="align:center"><img src="doc-files/timeline-advanced.svg" alt="Example timeline for an
- * on-demand stream followed by a live stream"> This case is the concatenation of the <a
+ * on-demand stream followed by a live stream">
- * href="#single-file">Single media file or on-demand stream</a> and <a href="#multi-period">Live
+ *
- * stream with multiple periods</a> cases. When playback of the on-demand stream ends, playback of
+ * <p>This case is the concatenation of the <a href="#single-file">Single media file or on-demand
- * the live stream will start from its default position near the live edge.
+ * stream</a> and <a href="#multi-period">Live stream with multiple periods</a> cases. When playback
+ * of the on-demand stream ends, playback of the live stream will start from its default position
+ * near the live edge.
  *
  * <h3 id="single-file-midrolls">On-demand stream with mid-roll ads</h3>
  *
  * <p style="align:center"><img src="doc-files/timeline-single-file-midrolls.svg" alt="Example
- * timeline for an on-demand stream with mid-roll ad groups"> This case includes mid-roll ad groups,
+ * timeline for an on-demand stream with mid-roll ad groups">
- * which are defined as part of the timeline's single period. The period can be queried for
+ *
- * information about the ad groups and the ads they contain.
+ * <p>This case includes mid-roll ad groups, which are defined as part of the timeline's single
+ * period. The period can be queried for information about the ad groups and the ads they contain.
  */
 public abstract class Timeline {
 

library/core/src/main/java/com/google/android/exoplayer2/util/EventLogger.java

@@ -318,12 +318,6 @@ public class EventLogger implements AnalyticsListener {
     logd(eventTime, "audioInputFormat", Format.toLogString(format));
   }
 
-  @Override
-  public void onAudioPositionAdvancing(EventTime eventTime, long playoutStartSystemTimeMs) {
-    long timeSincePlayoutStartMs = System.currentTimeMillis() - playoutStartSystemTimeMs;
-    logd(eventTime, "audioPositionAdvancing", "timeSincePlayoutStartMs=" + timeSincePlayoutStartMs);
-  }
-
   @Override
   public void onAudioUnderrun(
       EventTime eventTime, int bufferSize, long bufferSizeMs, long elapsedSinceLastFeedMs) {