Update javadoc to reflect removal of WebM container option

It seems fine to remove the documentation about the WebM case now we are only supporting unfragmented MP4, so that new users coming to this API aren't confused about how to set the container MIME type. PiperOrigin-RevId: 413611472
google · Dec 2, 2021 · 7e82225 · 7e82225
1 parent d53d51d
commit 7e82225
Showing 1 changed file with 9 additions and 31 deletions.
diff --git a/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java b/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java
@@ -26,8 +26,6 @@
 
 import android.content.Context;
 import android.graphics.Matrix;
-import android.media.MediaFormat;
-import android.media.MediaMuxer;
 import android.os.Handler;
 import android.os.Looper;
 import android.os.ParcelFileDescriptor;
@@ -298,18 +296,10 @@ public Builder setOutputMimeType(String outputMimeType) {
      * input. Supported values are:
      *
      * <ul>
-     *   <li>when the container MIME type is {@link MimeTypes#VIDEO_MP4}:
-     *       <ul>
-     *         <li>{@link MimeTypes#VIDEO_H263}
-     *         <li>{@link MimeTypes#VIDEO_H264}
-     *         <li>{@link MimeTypes#VIDEO_H265} from API level 24
-     *         <li>{@link MimeTypes#VIDEO_MP4V}
-     *       </ul>
-     *   <li>when the container MIME type is {@link MimeTypes#VIDEO_WEBM}:
-     *       <ul>
-     *         <li>{@link MimeTypes#VIDEO_VP8}
-     *         <li>{@link MimeTypes#VIDEO_VP9} from API level 24
-     *       </ul>
+     *   <li>{@link MimeTypes#VIDEO_H263}
+     *   <li>{@link MimeTypes#VIDEO_H264}
+     *   <li>{@link MimeTypes#VIDEO_H265} from API level 24
+     *   <li>{@link MimeTypes#VIDEO_MP4V}
      * </ul>
      *
      * @param videoMimeType The MIME type of the video samples in the output.
@@ -325,16 +315,9 @@ public Builder setVideoMimeType(String videoMimeType) {
      * input. Supported values are:
      *
      * <ul>
-     *   <li>when the container MIME type is {@link MimeTypes#VIDEO_MP4}:
-     *       <ul>
-     *         <li>{@link MimeTypes#AUDIO_AAC}
-     *         <li>{@link MimeTypes#AUDIO_AMR_NB}
-     *         <li>{@link MimeTypes#AUDIO_AMR_WB}
-     *       </ul>
-     *   <li>when the container MIME type is {@link MimeTypes#VIDEO_WEBM}:
-     *       <ul>
-     *         <li>{@link MimeTypes#AUDIO_VORBIS}
-     *       </ul>
+     *   <li>{@link MimeTypes#AUDIO_AAC}
+     *   <li>{@link MimeTypes#AUDIO_AMR_NB}
+     *   <li>{@link MimeTypes#AUDIO_AMR_WB}
      * </ul>
      *
      * @param audioMimeType The MIME type of the audio samples in the output.
@@ -404,7 +387,6 @@ public Builder setLooper(Looper looper) {
      * @throws NullPointerException If the {@link Context} has not been provided.
      * @throws IllegalStateException If both audio and video have been removed (otherwise the output
      *     would not contain any samples).
-     * @throws IllegalStateException If the muxer doesn't support the requested container MIME type.
      * @throws IllegalStateException If the muxer doesn't support the requested audio MIME type.
      * @throws IllegalStateException If the muxer doesn't support the requested video MIME type.
      */
@@ -559,9 +541,7 @@ public void setListener(Transformer.Listener listener) {
    * track types are ignored. For adaptive bitrate {@link MediaSource media sources}, the highest
    * bitrate video and audio streams are selected.
    *
-   * @param mediaItem The {@link MediaItem} to transform. The supported sample formats depend on the
-   *     {@link Muxer} and on the output container format. For the {@link FrameworkMuxer}, they are
-   *     described in {@link MediaMuxer#addTrack(MediaFormat)}.
+   * @param mediaItem The {@link MediaItem} to transform.
    * @param path The path to the output file.
    * @throws IllegalArgumentException If the path is invalid.
    * @throws IllegalStateException If this method is called from the wrong thread.
@@ -584,9 +564,7 @@ public void startTransformation(MediaItem mediaItem, String path) throws IOExcep
    * track types are ignored. For adaptive bitrate {@link MediaSource media sources}, the highest
    * bitrate video and audio streams are selected.
    *
-   * @param mediaItem The {@link MediaItem} to transform. The supported sample formats depend on the
-   *     {@link Muxer} and on the output container format. For the {@link FrameworkMuxer}, they are
-   *     described in {@link MediaMuxer#addTrack(MediaFormat)}.
+   * @param mediaItem The {@link MediaItem} to transform.
    * @param parcelFileDescriptor A readable and writable {@link ParcelFileDescriptor} of the output.
    *     The file referenced by this ParcelFileDescriptor should not be used before the
    *     transformation is completed. It is the responsibility of the caller to close the