Add missing `@param` tags

Dackka generates a warning if a method has at least one `@param` tag,
but not all of them are documented.

#minor-release

PiperOrigin-RevId: 513873453

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

@@ -809,6 +809,7 @@ public abstract class Timeline implements Bundleable {
      * adGroupIndex}, or {@link AdPlaybackState#AD_STATE_UNAVAILABLE} if not yet known.
      *
      * @param adGroupIndex The ad group index.
+     * @param adIndexInAdGroup The index of the ad in the ad group.
      * @return The state of the ad, or {@link AdPlaybackState#AD_STATE_UNAVAILABLE} if not yet
      *     known.
      */

library/common/src/main/java/com/google/android/exoplayer2/Tracks.java

@@ -293,6 +293,7 @@ public final class Tracks implements Bundleable {
    * Returns true if at least one track of type {@code trackType} is {@link
    * Group#isTrackSupported(int, boolean) supported}.
    *
+   * @param trackType The track type to query support for.
    * @param allowExceedsCapabilities Whether to consider the track as supported if it has a
    *     supported {@link Format#sampleMimeType MIME type}, but otherwise exceeds the advertised
    *     capabilities of the device. For example, a video track for which there's a corresponding

library/common/src/main/java/com/google/android/exoplayer2/util/Util.java

@@ -1492,6 +1492,7 @@ public final class Util {
    * Returns the playout duration of {@code mediaDuration} of media.
    *
    * @param mediaDuration The duration to scale.
+   * @param speed The factor by which playback is sped up.
    * @return The scaled duration, in the same units as {@code mediaDuration}.
    */
   public static long getPlayoutDurationForMediaDuration(long mediaDuration, float speed) {

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

@@ -125,6 +125,8 @@ public final class ExoPlaybackException extends PlaybackException {
    * Creates an instance of type {@link #TYPE_RENDERER}.
    *
    * @param cause The cause of the failure.
+   * @param rendererName The {@linkplain Renderer#getName() name} of the renderer in which the
+   *     failure occurred.
    * @param rendererIndex The index of the renderer in which the failure occurred.
    * @param rendererFormat The {@link Format} the renderer was using at the time of the exception,
    *     or null if the renderer wasn't using a {@link Format}.

library/core/src/main/java/com/google/android/exoplayer2/drm/OfflineLicenseHelper.java

@@ -97,6 +97,7 @@ public final class OfflineLicenseHelper {
    *     their own license URL.
    * @param forceDefaultLicenseUrl Whether to use {@code defaultLicenseUrl} for key requests that
    *     include their own license URL.
+   * @param dataSourceFactory A factory from which to obtain {@link DataSource} instances.
    * @param optionalKeyRequestParameters An optional map of parameters to pass as the last argument
    *     to {@link MediaDrm#getKeyRequest}. May be null.
    * @param eventDispatcher A {@link DrmSessionEventListener.EventDispatcher} used to distribute

library/core/src/main/java/com/google/android/exoplayer2/mediacodec/MediaCodecRenderer.java

@@ -362,6 +362,7 @@ public abstract class MediaCodecRenderer extends BaseRenderer {
 
   /**
    * @param trackType The {@link C.TrackType track type} that the renderer handles.
+   * @param codecAdapterFactory A factory for {@link MediaCodecAdapter} instances.
    * @param mediaCodecSelector A decoder selector.
    * @param enableDecoderFallback Whether to enable fallback to lower-priority decoders if decoder
    *     initialization fails. This may result in using a decoder that is less efficient or slower

library/core/src/main/java/com/google/android/exoplayer2/offline/SegmentDownloader.java

@@ -323,6 +323,7 @@ public abstract class SegmentDownloader<M extends FilterableManifest<M>> impleme
   /**
    * Loads and parses a manifest.
    *
+   * @param dataSource The source to use when loading the manifest.
    * @param dataSpec The manifest {@link DataSpec}.
    * @param removing Whether the manifest is being loaded as part of the download being removed.
    * @return The loaded manifest.

library/core/src/main/java/com/google/android/exoplayer2/trackselection/DefaultTrackSelector.java

@@ -2439,6 +2439,7 @@ public class DefaultTrackSelector extends MappingTrackSelector {
    *     renderer, track group and track (in that order).
    * @param rendererMixedMimeTypeAdaptationSupports The {@link AdaptiveSupport} for mixed MIME type
    *     adaptation for the renderer.
+   * @param params The parameters to use for the track selection.
    * @return The {@link ExoTrackSelection.Definition}s for the renderers. A null entry indicates no
    *     selection was made.
    * @throws ExoPlaybackException If an error occurs while selecting the tracks.

library/core/src/main/java/com/google/android/exoplayer2/trackselection/MappingTrackSelector.java

@@ -287,6 +287,7 @@ public abstract class MappingTrackSelector extends TrackSelector {
      *
      * @param rendererIndex The renderer index.
      * @param groupIndex The index of the track group.
+     * @param trackIndices The indices of the tracks in the track group for which to query support.
      * @return The {@link AdaptiveSupport}.
      */
     public @AdaptiveSupport int getAdaptiveSupport(

library/core/src/main/java/com/google/android/exoplayer2/trackselection/RandomTrackSelection.java

@@ -66,12 +66,15 @@ public final class RandomTrackSelection extends BaseTrackSelection {
   private int selectedIndex;
 
   /**
+   * Creates a new instance.
+   *
    * @param group The {@link TrackGroup}. Must not be null.
    * @param tracks The indices of the selected tracks within the {@link TrackGroup}. Must not be
    *     null or empty. May be in any order.
+   * @param type The {@link Type} of this track selection.
    * @param random A source of random numbers.
    */
-  public RandomTrackSelection(TrackGroup group, int[] tracks, int type, Random random) {
+  public RandomTrackSelection(TrackGroup group, int[] tracks, @Type int type, Random random) {
     super(group, tracks, type);
     this.random = random;
     selectedIndex = random.nextInt(length);

library/core/src/main/java/com/google/android/exoplayer2/video/DecoderVideoRenderer.java

@@ -648,6 +648,7 @@ public abstract class DecoderVideoRenderer extends BaseRenderer {
    *
    * <p>The default implementation does not allow decoder reuse.
    *
+   * @param decoderName The name of the decoder.
    * @param oldFormat The previous format.
    * @param newFormat The new format.
    * @return The result of the evaluation.

library/dash/src/main/java/com/google/android/exoplayer2/source/dash/DashChunkSource.java

@@ -48,6 +48,8 @@ public interface DashChunkSource extends ChunkSource {
      *     if unknown.
      * @param enableEventMessageTrack Whether to output an event message track.
      * @param closedCaptionFormats The {@link Format Formats} of closed caption tracks to be output.
+     * @param playerEmsgHandler The track output to write emsg messages to, or null if emsgs
+     *     shouldn't be written.
      * @param transferListener The transfer listener which should be informed of any data transfers.
      *     May be null if no listener is available.
      * @param playerId The {@link PlayerId} of the player using this chunk source.
@@ -73,8 +75,9 @@ public interface DashChunkSource extends ChunkSource {
    * Updates the manifest.
    *
    * @param newManifest The new manifest.
+   * @param newPeriodIndex The index of the period covered by {@code newManifest}.
    */
-  void updateManifest(DashManifest newManifest, int periodIndex);
+  void updateManifest(DashManifest newManifest, int newPeriodIndex);
 
   /**
    * Updates the track selection.

library/datasource/src/main/java/com/google/android/exoplayer2/upstream/DataSourceUtil.java

@@ -54,6 +54,7 @@ public final class DataSourceUtil {
    * array containing the read data.
    *
    * @param dataSource The source from which to read.
+   * @param length The number of bytes to read.
    * @return The read data.
    * @throws IOException If an error occurs reading from the source.
    * @throws IllegalStateException If the end of the source was reached before {@code length} bytes

library/datasource/src/main/java/com/google/android/exoplayer2/upstream/DefaultDataSource.java

@@ -145,6 +145,7 @@ public final class DefaultDataSource implements DataSource {
    * Constructs a new instance, optionally configured to follow cross-protocol redirects.
    *
    * @param context A context.
+   * @param allowCrossProtocolRedirects Whether to allow cross-protocol redirects.
    */
   public DefaultDataSource(Context context, boolean allowCrossProtocolRedirects) {
     this(

library/effect/src/main/java/com/google/android/exoplayer2/effect/BitmapOverlay.java

@@ -89,6 +89,8 @@ public abstract class BitmapOverlay extends TextureOverlay {
   /**
    * Creates a {@link BitmapOverlay} that shows the {@code overlayBitmap} in the same position and
    * size throughout the whole video.
+   *
+   * @param overlayBitmap The bitmap to overlay on the video.
    */
   public static BitmapOverlay createStaticBitmapOverlay(Bitmap overlayBitmap) {
     return new BitmapOverlay() {
@@ -103,6 +105,7 @@ public abstract class BitmapOverlay extends TextureOverlay {
    * Creates a {@link BitmapOverlay} that shows the {@code overlayBitmap} in the same {@link
    * OverlaySettings} throughout the whole video.
    *
+   * @param overlayBitmap The bitmap to overlay on the video.
    * @param overlaySettings The {@link OverlaySettings} configuring how the overlay is displayed on
    *     the frames.
    */

library/effect/src/main/java/com/google/android/exoplayer2/effect/TextOverlay.java

@@ -54,6 +54,7 @@ public abstract class TextOverlay extends BitmapOverlay {
    * Creates a {@link TextOverlay} that shows the {@code overlayText} with the same {@link
    * OverlaySettings} throughout the whole video.
    *
+   * @param overlayText The text to overlay on the video.
    * @param overlaySettings The {@link OverlaySettings} configuring how the overlay is displayed on
    *     the frames.
    */

library/extractor/src/main/java/com/google/android/exoplayer2/extractor/FlacFrameReader.java

@@ -130,6 +130,7 @@ public final class FlacFrameReader {
    * there is no guarantee on the peek position.
    *
    * @param input Input stream to get the sample number from (starting from the read position).
+   * @param flacStreamMetadata The FLAC metadata of the stream.
    * @return The frame first sample number.
    * @throws ParserException If an error occurs parsing the sample number.
    * @throws IOException If peeking from the input fails.

library/extractor/src/main/java/com/google/android/exoplayer2/text/webvtt/WebvttCssStyle.java

@@ -150,12 +150,13 @@ public final class WebvttCssStyle {
    *   <li>Universal selector matching scores 1.
    * </ul>
    *
+   * <p>See also <a href="https://www.w3.org/TR/CSS2/cascade.html">CSS Cascading</a>.
+   *
    * @param id The id of the cue if present, {@code null} otherwise.
    * @param tag Name of the tag, {@code null} if it refers to the entire cue.
    * @param classes An array containing the classes the tag belongs to. Must not be null.
    * @param voice Annotated voice if present, {@code null} otherwise.
    * @return The score of the match, zero if there is no match.
-   * @see <a href="https://www.w3.org/TR/CSS2/cascade.html">CSS Cascading</a>
    */
   public int getSpecificityScore(
       @Nullable String id, @Nullable String tag, Set<String> classes, @Nullable String voice) {

library/hls/src/main/java/com/google/android/exoplayer2/source/hls/HlsMediaPeriod.java

@@ -102,13 +102,17 @@ public final class HlsMediaPeriod implements MediaPeriod, HlsPlaylistTracker.Pla
    *     be null if no listener is available.
    * @param drmSessionManager The {@link DrmSessionManager} to acquire {@link DrmSession
    *     DrmSessions} with.
+   * @param drmEventDispatcher A {@link DrmSessionEventListener.EventDispatcher} used to distribute
+   *     DRM-related events.
    * @param loadErrorHandlingPolicy A {@link LoadErrorHandlingPolicy}.
    * @param eventDispatcher A dispatcher to notify of events.
    * @param allocator An {@link Allocator} from which to obtain media buffer allocations.
    * @param compositeSequenceableLoaderFactory A factory to create composite {@link
    *     SequenceableLoader}s for when this media source loads data from multiple streams.
    * @param allowChunklessPreparation Whether chunkless preparation is allowed.
+   * @param metadataType The type of metadata to extract from the period.
    * @param useSessionKeys Whether to use #EXT-X-SESSION-KEY tags.
+   * @param playerId The ID of the current player.
    */
   public HlsMediaPeriod(
       HlsExtractorFactory extractorFactory,

library/hls/src/main/java/com/google/android/exoplayer2/source/hls/playlist/HlsMediaPlaylist.java

@@ -460,16 +460,20 @@ public final class HlsMediaPlaylist extends HlsPlaylist {
   public final ServerControl serverControl;
 
   /**
+   * Constructs an instance.
+   *
    * @param playlistType See {@link #playlistType}.
    * @param baseUri See {@link #baseUri}.
    * @param tags See {@link #tags}.
    * @param startOffsetUs See {@link #startOffsetUs}.
+   * @param preciseStart See {@link #preciseStart}.
    * @param startTimeUs See {@link #startTimeUs}.
    * @param hasDiscontinuitySequence See {@link #hasDiscontinuitySequence}.
    * @param discontinuitySequence See {@link #discontinuitySequence}.
    * @param mediaSequence See {@link #mediaSequence}.
    * @param version See {@link #version}.
    * @param targetDurationUs See {@link #targetDurationUs}.
+   * @param partTargetDurationUs See {@link #partTargetDurationUs}.
    * @param hasIndependentSegments See {@link #hasIndependentSegments}.
    * @param hasEndTag See {@link #hasEndTag}.
    * @param hasProgramDateTime See {@link #hasProgramDateTime}.

testutils/src/main/java/com/google/android/exoplayer2/testutil/Action.java

@@ -1096,7 +1096,10 @@ public abstract class Action {
     private final Runnable runnable;
 
     /**
+     * Constructs an instance.
+     *
      * @param tag A tag to use for logging.
+     * @param runnable The runnable to run.
      */
     public ExecuteRunnable(@Size(max = 23) String tag, Runnable runnable) {
       super(tag, "ExecuteRunnable");

testutils/src/main/java/com/google/android/exoplayer2/testutil/ActionSchedule.java

@@ -347,7 +347,7 @@ public final class ActionSchedule {
     }
 
     /**
-     * Schedules a set media items action to be executed.
+     * Schedules a set media source actions to be executed.
      *
      * @param mediaItemIndex The media item index to start playback from or {@link C#INDEX_UNSET} if
      *     the playback position should not be reset.
@@ -355,6 +355,7 @@ public final class ActionSchedule {
      *     C#TIME_UNSET} is passed the default position is used. In any case, if {@code
      *     mediaItemIndex} is set to {@link C#INDEX_UNSET} the position is not reset at all and this
      *     parameter is ignored.
+     * @param sources The media sources to be set on the player.
      * @return The builder, for convenience.
      */
     @CanIgnoreReturnValue
@@ -363,9 +364,10 @@ public final class ActionSchedule {
     }
 
     /**
-     * Schedules a set media items action to be executed.
+     * Schedules a set media sources action to be executed.
      *
      * @param resetPosition Whether the playback position should be reset.
+     * @param sources The media sources to be set on the player.
      * @return The builder, for convenience.
      */
     @CanIgnoreReturnValue
@@ -488,6 +490,7 @@ public final class ActionSchedule {
     /**
      * Schedules sending a {@link PlayerMessage}.
      *
+     * @param target A message target.
      * @param positionMs The position in the current media item at which the message should be sent,
      *     in milliseconds.
      * @return The builder, for convenience.

testutils/src/main/java/com/google/android/exoplayer2/testutil/BitmapPixelTestUtil.java

@@ -104,6 +104,8 @@ public class BitmapPixelTestUtil {
   /**
    * Returns a solid {@link Bitmap} with every pixel having the same color.
    *
+   * @param width The width of image to create, in pixels.
+   * @param height The height of image to create, in pixels.
    * @param color An RGBA color created by {@link Color}.
    */
   public static Bitmap createArgb8888BitmapWithSolidColor(int width, int height, int color) {

testutils/src/main/java/com/google/android/exoplayer2/testutil/ExtractorAsserts.java

@@ -358,6 +358,9 @@ public final class ExtractorAsserts {
    *
    * <p>The output of the extractor is compared against prerecorded dump files.
    *
+   * @param factory An {@link ExtractorFactory} which creates instances of the {@link Extractor}
+   *     class which is to be tested.
+   * @param file The input file to pass to the extractor.
    * @param assertionConfig Details of how to read and process the source and dump files.
    * @param simulationConfig Details on the environment to simulate and behaviours to assert.
    * @throws IOException If reading from the input fails.

testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeMediaPeriod.java

@@ -373,6 +373,7 @@ public class FakeMediaPeriod implements MediaPeriod {
   /**
    * Creates a new {@link FakeSampleStream}.
    *
+   * @param allocator An {@link Allocator} from which to obtain media buffer allocations.
    * @param mediaSourceEventDispatcher A {@link MediaSourceEventListener.EventDispatcher} to notify
    *     of media events.
    * @param drmSessionManager A {@link DrmSessionManager} for DRM interactions.

testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeMediaSource.java

@@ -337,6 +337,7 @@ public class FakeMediaSource extends BaseMediaSource {
    * @param allocator An {@link Allocator} from which to obtain media buffer allocations.
    * @param mediaSourceEventDispatcher An {@link MediaSourceEventListener.EventDispatcher} to
    *     dispatch media source events.
+   * @param drmSessionManager A {@link DrmSessionManager} to allow DRM interactions.
    * @param drmEventDispatcher An {@link MediaSourceEventListener.EventDispatcher} to dispatch DRM
    *     events.
    * @param transferListener The transfer listener which should be informed of any data transfers.

testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeTimeline.java

@@ -391,6 +391,7 @@ public final class FakeTimeline extends Timeline {
   /**
    * Creates a fake timeline with the given window definitions.
    *
+   * @param manifests The manifests of the windows.
    * @param windowDefinitions A list of {@link TimelineWindowDefinition}s.
    */
   public FakeTimeline(Object[] manifests, TimelineWindowDefinition... windowDefinitions) {
@@ -401,6 +402,8 @@ public final class FakeTimeline extends Timeline {
    * Creates a fake timeline with the given window definitions and {@link
    * com.google.android.exoplayer2.source.ShuffleOrder}.
    *
+   * @param manifests The manifests of the windows.
+   * @param shuffleOrder A shuffle ordering for the windows.
    * @param windowDefinitions A list of {@link TimelineWindowDefinition}s.
    */
   public FakeTimeline(

testutils/src/main/java/com/google/android/exoplayer2/testutil/MediaSourceTestRunner.java

@@ -150,6 +150,7 @@ public class MediaSourceTestRunner {
    * playback thread, asserting that a non-null {@link MediaPeriod} is returned.
    *
    * @param periodId The id of the period to create.
+   * @param startPositionUs The expected start position, in microseconds.
    * @return The created {@link MediaPeriod}.
    */
   public MediaPeriod createPeriod(final MediaPeriodId periodId, long startPositionUs) {

testutils/src/main/java/com/google/android/exoplayer2/testutil/TimelineAsserts.java

@@ -48,6 +48,7 @@ public final class TimelineAsserts {
   /**
    * Asserts that window tags are set correctly.
    *
+   * @param timeline The timeline to read actual window tags from.
    * @param expectedWindowTags A list of expected window tags. If a tag is unknown or not important
    *     {@code null} can be passed to skip this window.
    */