- 11 May, 2023 15 commits
-
-
#minor-release PiperOrigin-RevId: 530935437
ibaker committed -
PiperOrigin-RevId: 530888319
tofunmi committed -
PiperOrigin-RevId: 530878135
sheenachhabra committed -
PiperOrigin-RevId: 530711523
Googler committed -
To use the InAppMuxer, the client needs to pass InAppMuxer Factory. PiperOrigin-RevId: 530684007
sheenachhabra committed -
Verified by running all the tests in mh (excluding the analysis tests) on a pixel 7 pro PiperOrigin-RevId: 530655237
tofunmi committed -
Tone-mapping an HDR video with MediaCodec on sm-s908u1 was timing out. The reason for that is that the decoder was dropping frames, and the ExternalTextureManager was therefore never propagating the end-of-stream signal. There was already a workaround for a similar issue but restricted to sm-f936b. Removed the model check as the bug is probably present on more devices. PiperOrigin-RevId: 530639437
kimvde committed -
Exceptions thrown when switching AssetLoader were not reported through the error listener. Therefore, the resources were not released and the export was not ending. PiperOrigin-RevId: 530626300
kimvde committed -
PiperOrigin-RevId: 530624195
tofunmi committed -
Allow use of H265/H264 codecs for Dolby Vision video. Also, reflow ExoPlayer code to use this new utility class PiperOrigin-RevId: 530619388
huangdarwin committed -
If tone-mapping was requested for a device supporting HDR encoding, isToneMapped was false in VideoSamplePipeline.EncoderWrapper. This was causing the encoder to expect HDR. Also did some renamings to improve readability PiperOrigin-RevId: 530584010
kimvde committed -
similar error to https://github.com/google/ExoPlayer/commit/2c4a567c74088c639d699eeefe139c38e254e230L `registerInputStream` sets the textureManager, so reordering makes sure the textureManager is set before you set the frame info. This is important for texture input, where the frame info provides the width and height. PiperOrigin-RevId: 530579584
tofunmi committed -
PiperOrigin-RevId: 530578549
tofunmi committed -
Adding items to an empty playlist is slightly different from adding items to a non-empty playlist, because the former usually requires to handle a change in the current item, position and playback state, while the latter is not expected to affect the current item, position or state. The current ExoPlayer and SimpleBasePlayer code doesn't account for this difference, leading to inconsistent behavior between setMediaItem(s) and addMediaItem(s) when called on an empty playlist. PiperOrigin-RevId: 530549928
tonihei committed -
PiperOrigin-RevId: 530314222
andrewlewis committed
-
- 08 May, 2023 12 commits
-
-
For DefaultVideoFrameProcessorVideoFrameRenderingTest: `registerInputStream` sets the textureManager, so reordering makes sure the textureManager is set before you set the frame info. This is important for texture input, where the frame info provides the width and height. PiperOrigin-RevId: 529753404
tofunmi committed -
`VERSION_INT` is quite long with several sections, and it's easy to make a mistake when updating it - this should help since it checks it against `VERSION`, which is more easily human readable/writable. PiperOrigin-RevId: 529747023
ibaker committed -
#minor-release PiperOrigin-RevId: 529743667
bachinger committed -
#minor-release PiperOrigin-RevId: 529742339
bachinger committed -
Minimal fix for HDR test postsubmit breakage PiperOrigin-RevId: 529734521
huangdarwin committed -
This setting is buggy on some devices (for example sm-n960f) PiperOrigin-RevId: 529712770
kimvde committed -
#minor-release PiperOrigin-RevId: 529710928
bachinger committed -
#minor-release PiperOrigin-RevId: 529708020
michaelkatz committed -
PiperOrigin-RevId: 529693415
sheenachhabra committed -
PiperOrigin-RevId: 529685584
claincly committed -
PiperOrigin-RevId: 529632886
claincly committed -
PiperOrigin-RevId: 529624205
claincly committed
-
- 04 May, 2023 1 commit
-
-
This change: * Adds missing `@OptIn` annotation to demo app's `ErrorMessageProvider` * Switches from `Util.SDK_INT` to `Build.VERSION.SDK_INT` in `SampleChooserActivity` (`PlayerActivity` is already using this). This code hasn't changed recently, and it doesn't fail on the `release` branch, but it failed when I checked the `main` branch just now - so I assume lint has updated to detect more cases where unstable APIs are being used without opt-in. I suspect the difference is due to different Android Gradle Plugin versions between the branches. #minor-release PiperOrigin-RevId: 529111669
ibaker committed
-
- 03 May, 2023 12 commits
-
-
A list was being accessed from one thread when it wasn't guaranteed to be empty. PiperOrigin-RevId: 529102141
huangdarwin committed -
PiperOrigin-RevId: 529069808
tofunmi committed -
* Implement RendererCapabilities.Listener in DefaultTrackSelector. * Add new methods TrackSelector.invalidateForRendererCapabilitiesChange and TrackSelector.InvalidateListener.onRendererCapabilitiesChanged. * Add new field allowInvalidateSelectionsOnRendererCapabilitiesChange to DefaultTrackSelector.Parameter to allow opt-in of the renderer capabilities detection feature. * Add logics of triggering track reselection when renderer capabilities change. PiperOrigin-RevId: 529067433
tianyifeng committed -
When we created androidTests, in the past, they always had a URI pointing to a resource, therefore we always had a URI scheme. With texture input, this will not longer be the case (EditedMediaItems's may have URI.EMPTY, which have a null scheme) so we need to check for this so tests don't falsely fail. PiperOrigin-RevId: 528848411
tofunmi committed -
The existing logic to drop (actually fail on) 0 sized samples seems no op if if 2 out of 10 samples are of size 0. Checked same scenario with MediaMuxer where 1. If input file has 300 samples. 2. Make every 5th sample as an empty byte buffer. 3. Output file is generated without error. 4. Output file has 240 samples. 5. Exoplayer is able to play output file (blurry). The new change is in line with MediaMuxer behaviour. PiperOrigin-RevId: 528798046
sheenachhabra committed -
Systems accepting URIs should treat schemes as case-insensitive ([RFC 3986 Section 3.1](https://www.rfc-editor.org/rfc/rfc3986#section-3.1)): > An implementation should accept uppercase letters as equivalent to > lowercase in scheme names (e.g., allow "HTTP" as well as "http") for > the sake of robustness PiperOrigin-RevId: 528735287
ibaker committed -
PiperOrigin-RevId: 528734663
claincly committed -
By default Android Studio will name the project based on the root directory it's opened from. This gives a consistent (and clear) name regardless of what root directory the project is located in on the filesystem. Explicitly defining `rootProject.name` is recommended here: https://docs.gradle.org/current/userguide/multi_project_builds.html#naming_recommendations PiperOrigin-RevId: 528729078
ibaker committed -
This means that comments like `//copybara:media3-only` are now detected (no space between `//` and `copybara`) which will ensure that lines like this are correctly transformed for the media3 and exoplayer2 GitHub repos: https://github.com/google/ExoPlayer/blob/aa4e0080147fe76a6c0cca40b7eccdf45eb58028/library/effect/build.gradle#L33 PiperOrigin-RevId: 527919649
ibaker committed -
Issue: androidx/media#319 PiperOrigin-RevId: 527891646
michaelkatz committed -
Also make FinalShaderProgramWrapper always receive internal texture. This means it does not sample from a input texture, and its input color is always linear, hence the input type does not matter. PiperOrigin-RevId: 527869045
claincly committed -
https://github.com/google/ExoPlayer/commit/5a62d1bc25e921df363eb741b82b00a7c87dc94f
*** Original commit *** Effect: glFlush instead of glFinish on tex output This is much faster (~2-3x) than glFlush. While there's a risk that GL commands queued to the GL server may not be complete by the time non-GL commands access the texture, this should be unlikely as we only access the texture from GL. If we see stability issues in the future, we can reconsider and move this back to glFinish (or GL synchronization mechanisms like fences, which are more complex) *** PiperOrigin-RevId: 527848094
tofunmi committed
-