Add bmpExtractor to DefaultExtractorsFactory

PiperOrigin-RevId: 563076972
This commit is contained in:
tofunmi 2023-09-06 05:39:21 -07:00 committed by Copybara-Service
parent ef45c0fe5d
commit 88b114a997
3 changed files with 23 additions and 4 deletions

View file

@ -56,6 +56,7 @@ public final class FileTypes {
* <li>{@link #AVI}
* <li>{@link #PNG}
* <li>{@link #WEBP}
* <li>{@link #BMP}
* </ul>
*/
@Documented
@ -63,7 +64,7 @@ public final class FileTypes {
@Target(TYPE_USE)
@IntDef({
UNKNOWN, AC3, AC4, ADTS, AMR, FLAC, FLV, MATROSKA, MP3, MP4, OGG, PS, TS, WAV, WEBVTT, JPEG,
MIDI, AVI, PNG, WEBP
MIDI, AVI, PNG, WEBP, BMP
})
public @interface Type {}
@ -127,6 +128,9 @@ public final class FileTypes {
/** File type for the WEBP format. */
public static final int WEBP = 18;
/** File type for the BMP format. */
public static final int BMP = 19;
@VisibleForTesting /* package */ static final String HEADER_CONTENT_TYPE = "Content-Type";
private static final String EXTENSION_AC3 = ".ac3";
@ -164,6 +168,8 @@ public final class FileTypes {
private static final String EXTENSION_AVI = ".avi";
private static final String EXTENSION_PNG = ".png";
private static final String EXTENSION_WEBP = ".webp";
private static final String EXTENSION_BMP = ".bmp";
private static final String EXTENSION_DIB = ".dib";
private FileTypes() {}
@ -233,6 +239,8 @@ public final class FileTypes {
return FileTypes.PNG;
case MimeTypes.IMAGE_WEBP:
return FileTypes.WEBP;
case MimeTypes.IMAGE_BMP:
return FileTypes.BMP;
default:
return FileTypes.UNKNOWN;
}
@ -304,6 +312,8 @@ public final class FileTypes {
return FileTypes.PNG;
} else if (filename.endsWith(EXTENSION_WEBP)) {
return FileTypes.WEBP;
} else if (filename.endsWith(EXTENSION_BMP) || filename.endsWith(EXTENSION_DIB)) {
return FileTypes.BMP;
} else {
return FileTypes.UNKNOWN;
}

View file

@ -30,6 +30,7 @@ import androidx.media3.common.util.TimestampAdjuster;
import androidx.media3.common.util.UnstableApi;
import androidx.media3.extractor.amr.AmrExtractor;
import androidx.media3.extractor.avi.AviExtractor;
import androidx.media3.extractor.bmp.BmpExtractor;
import androidx.media3.extractor.flac.FlacExtractor;
import androidx.media3.extractor.flv.FlvExtractor;
import androidx.media3.extractor.jpeg.JpegExtractor;
@ -89,6 +90,7 @@ import java.util.concurrent.atomic.AtomicBoolean;
* <li>JPEG ({@link JpegExtractor})
* <li>PNG ({@link PngExtractor})
* <li>WEBP ({@link WebpExtractor})
* <li>BMP ({@link BmpExtractor})
* <li>MIDI, if available, the MIDI extension's {@code androidx.media3.decoder.midi.MidiExtractor}
* is used.
* </ul>
@ -121,7 +123,8 @@ public final class DefaultExtractorsFactory implements ExtractorsFactory {
FileTypes.MIDI,
FileTypes.JPEG,
FileTypes.PNG,
FileTypes.WEBP
FileTypes.WEBP,
FileTypes.BMP
};
private static final ExtensionLoader FLAC_EXTENSION_LOADER =
@ -518,6 +521,9 @@ public final class DefaultExtractorsFactory implements ExtractorsFactory {
case FileTypes.WEBP:
extractors.add(new WebpExtractor());
break;
case FileTypes.BMP:
extractors.add(new BmpExtractor());
break;
case FileTypes.WEBVTT:
case FileTypes.UNKNOWN:
default:

View file

@ -22,6 +22,7 @@ import android.net.Uri;
import androidx.media3.common.MimeTypes;
import androidx.media3.extractor.amr.AmrExtractor;
import androidx.media3.extractor.avi.AviExtractor;
import androidx.media3.extractor.bmp.BmpExtractor;
import androidx.media3.extractor.flac.FlacExtractor;
import androidx.media3.extractor.flv.FlvExtractor;
import androidx.media3.extractor.jpeg.JpegExtractor;
@ -78,7 +79,8 @@ public final class DefaultExtractorsFactoryTest {
AviExtractor.class,
JpegExtractor.class,
PngExtractor.class,
WebpExtractor.class)
WebpExtractor.class,
BmpExtractor.class)
.inOrder();
}
@ -123,7 +125,8 @@ public final class DefaultExtractorsFactoryTest {
AviExtractor.class,
JpegExtractor.class,
PngExtractor.class,
WebpExtractor.class)
WebpExtractor.class,
BmpExtractor.class)
.inOrder();
}