loliball/media3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Move MediaMetricsListener creation to static constructor method.

Browse Source 
This allows to check if the media metrics service is available outside
the actual constructor and to fail gracefully if it is missing.

PiperOrigin-RevId: 412232425
This commit is contained in:
tonihei 2021-11-25 10:01:47 +00:00
parent b7f0071e8c
commit 965062857d
1 changed files with 19 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/analytics/MediaMetricsListener.java
View File

@ -16,7 +16,6 @@
package androidx.media3.exoplayer.analytics; package androidx.media3.exoplayer.analytics;
import static androidx.media3.common.util.Assertions.checkNotNull; import static androidx.media3.common.util.Assertions.checkNotNull;
import static androidx.media3.common.util.Assertions.checkStateNotNull;
import static androidx.media3.common.util.Util.castNonNull; import static androidx.media3.common.util.Util.castNonNull;
import android.annotation.SuppressLint; import android.annotation.SuppressLint;
@ -92,6 +91,23 @@ import org.checkerframework.checker.nullness.qual.RequiresNonNull;
public final class MediaMetricsListener public final class MediaMetricsListener
implements AnalyticsListener, PlaybackSessionManager.Listener { implements AnalyticsListener, PlaybackSessionManager.Listener {
/**
* Creates a media metrics listener.
*
* @param context A context.
* @return The {@link MediaMetricsListener}, or null if the {@link Context#MEDIA_METRICS_SERVICE
* media metrics service} isn't available.
*/
@Nullable
public static MediaMetricsListener create(Context context) {
@Nullable
MediaMetricsManager mediaMetricsManager =
(MediaMetricsManager) context.getSystemService(Context.MEDIA_METRICS_SERVICE);
return mediaMetricsManager == null
? null
: new MediaMetricsListener(context, mediaMetricsManager.createPlaybackSession());
}
private final Context context; private final Context context;
private final PlaybackSessionManager sessionManager; private final PlaybackSessionManager sessionManager;
private final PlaybackSession playbackSession; private final PlaybackSession playbackSession;
@ -124,15 +140,12 @@ public final class MediaMetricsListener
* *
* @param context A {@link Context}. * @param context A {@link Context}.
*/ */
public MediaMetricsListener(Context context) { private MediaMetricsListener(Context context, PlaybackSession playbackSession) {
context = context.getApplicationContext(); context = context.getApplicationContext();
this.context = context; this.context = context;
this.playbackSession = playbackSession;
window = new Timeline.Window(); window = new Timeline.Window();
period = new Timeline.Period(); period = new Timeline.Period();
MediaMetricsManager mediaMetricsManager =
checkStateNotNull(
(MediaMetricsManager) context.getSystemService(Context.MEDIA_METRICS_SERVICE));
playbackSession = mediaMetricsManager.createPlaybackSession();
startTimeMs = SystemClock.elapsedRealtime(); startTimeMs = SystemClock.elapsedRealtime();
currentPlaybackState = PlaybackStateEvent.STATE_NOT_STARTED; currentPlaybackState = PlaybackStateEvent.STATE_NOT_STARTED;
currentNetworkType = NetworkEvent.NETWORK_TYPE_UNKNOWN; currentNetworkType = NetworkEvent.NETWORK_TYPE_UNKNOWN;