SDK / exoplayer

PiperOrigin-RevId: 383255572

PiperOrigin-RevId: 383251000

PiperOrigin-RevId: 383245932

This matches the Javadoc better.

PiperOrigin-RevId: 383228021

* @ImaAdState is not a TYPE_USE annotation, so should appear before any modifiers and after Javadocs.
  (see go/java-style#s4.8.5-annotations)

This CL looks good? Just LGTM and Approve it!
This CL doesn’t look good? This is what you can do:
* Suggest a fix on the CL (go/how-to-suggest-fix).
* Revert this CL, by replying "REVERT: <provide reason>"
* File a bug under go/error-prone-bug for category ErrorProneStyle if the change looks generally problematic.
* Revert this CL and not get a CL that cleans up these paths in the future by
replying "BLOCKLIST: <provide reason>". This is not reversible! We recommend to
opt out the respective paths in your CL Robot configuration instead:
go/clrobot-opt-out.

This CL was generated by CL Robot - a tool that cleans up code findings
(go/clrobot). The affected code paths have been enabled for CL Robot in //depot/google3/java/com/google/android/libraries/media/METADATA which is reachable following include_presubmits from //depot/google3/third_party/java_src/android_libs/media/METADATA.
Anything wrong with the signup? File a bug at go/clrobot-bug.

#codehealth

PiperOrigin-RevId: 383121587

And assign error codes to all callers.

PiperOrigin-RevId: 383118513

This isn't needed now we've updated to Robolectric 4.6

PiperOrigin-RevId: 383109343

The bug fix seems potentially serious and worth having:
https://github.com/robolectric/robolectric/releases/tag/robolectric-4.6.1

PiperOrigin-RevId: 383096745

PiperOrigin-RevId: 382766969

If a test expects an error they can use runUntilError()

PiperOrigin-RevId: 382765060

Issue: #9005
PiperOrigin-RevId: 382765045

MediaItem.mediaId used to default to the content URI, but this changed:
https://github.com/google/ExoPlayer/commit/cc26a92e070db8963738beed423bb4699762877f

Before the mediaId change linked above, a playlist of different content
all with the same ad URI would play the ads for every item. After the
change the ad would only play once (because mediaId == "" for every
item, so they're all the same). This change restores roughly the
original behaviour by always considering both mediaId and the content
URI.

#minor-release

Issue: #9106
PiperOrigin-RevId: 382763618

#minor-release

PiperOrigin-RevId: 382763308

The serialization scheme used here is custom, it doesn't need
to be compatible with emsg-v0 or emsg-v1 (since
https://github.com/google/ExoPlayer/commit/97183ef55866170807910cd626264d82d41d46d4).

This means that C.TIME_UNSET will propagate correctly through the
serialization.

#minor-release

Issue: #9123
PiperOrigin-RevId: 382762873

The tests now pass without @DoNotInstrument

I had to tweak the jetification settings due to a bug in the jetifier
shipped with the Android Gradle Plugin. More details:
https://github.com/robolectric/robolectric/issues/6521#issuecomment-851736355

PiperOrigin-RevId: 382757006

This is an abstract class so it can only be constructed from the context
of a subclass anyway.

PiperOrigin-RevId: 382756293

Without this annotation it seems that `SimpleExoPlayer` effectively
'un-deprecates' the method, specifically:
* A usage of these methods isn't flagged by Android Studio if the
  declared type is `SimpleExoPlayer` (up-casting to e.g.
  `ExoPlayer.VideoComponent` results in the warning showing up).
* The `SimpleExoPlayer` javadoc doesn't mention this method is
  deprecated:
  https://exoplayer.dev/doc/reference/com/google/android/exoplayer2/SimpleExoPlayer.html#addVideoListener(com.google.android.exoplayer2.video.VideoListener)
* The Metalava API output for `SimpleExoPlayer` doesn't show these
  methods as deprecated.

PiperOrigin-RevId: 382756174

PiperOrigin-RevId: 382710409

- Use `PlaybackException.ErrorCode` IntDef for `DataSourceException` error code
- Deprecate `DataSourceException.POSITION_OUT_OF_RANGE`
- All other changes are related to replacing the deprecated constant and
  constructor

PiperOrigin-RevId: 382683522

Also remove the method for creating a TYPE_RENDERER ExoPlaybackException
with unknown renderer name and index.

PiperOrigin-RevId: 382589655

PiperOrigin-RevId: 382551642

PiperOrigin-RevId: 382527115

In order to use it for error code assignment. Note that these DRM errors
could be thrown on API < 18 (from MediaCodec.CryptoException), which is
the reason we can't put the mapping in FrameworkMediaDrm.

PiperOrigin-RevId: 382512414

PiperOrigin-RevId: 382508442

It was impossible to call because it was a non-static method in a class
with a private constructor.

PiperOrigin-RevId: 382504128

These guarantees were documented for `EventListener` and
hold as well for Listener but were not documented.

PiperOrigin-RevId: 382498188

PiperOrigin-RevId: 382494191

Remove experimentalSetSkipAndContinueIfSampleTooLarge() api

PiperOrigin-RevId: 382383498

PlayerView and StyledPlayerView handled this in
onVideoSizeChanged but it can be omitted.
(e.g. if the player is MediaController)

PiperOrigin-RevId: 382340927

PiperOrigin-RevId: 382311518

Will be needed for error code assignment.

PiperOrigin-RevId: 382305328

PiperOrigin-RevId: 382297051

PiperOrigin-RevId: 382279955

PiperOrigin-RevId: 382277352

This affects the setter, getter and callback, as well as various
media 3 methods.

PiperOrigin-RevId: 382275672

PiperOrigin-RevId: 382139109

#minor-release

Issue:#7608
PiperOrigin-RevId: 382081687

PiperOrigin-RevId: 381941053

PiperOrigin-RevId: 381852972

PiperOrigin-RevId: 381852092