Commit 33af7a45 by tonihei Committed by kim-vde

Don't log mediaId in EventLogger and DefaultMediaSourceFactory.

The media id defaults to the URI that shouldn't be logged to logcat.

PiperOrigin-RevId: 324770157
parent ea01489c
...@@ -381,11 +381,7 @@ public final class DefaultMediaSourceFactory implements MediaSourceFactory { ...@@ -381,11 +381,7 @@ public final class DefaultMediaSourceFactory implements MediaSourceFactory {
} }
AdsLoader adsLoader = adSupportProvider.getAdsLoader(mediaItem.playbackProperties.adTagUri); AdsLoader adsLoader = adSupportProvider.getAdsLoader(mediaItem.playbackProperties.adTagUri);
if (adsLoader == null) { if (adsLoader == null) {
Log.w( Log.w(TAG, "Playing media without ads. No AdsLoader for provided adTagUri");
TAG,
String.format(
"Playing media without ads. No AdsLoader for media item with mediaId '%s'.",
mediaItem.mediaId));
return mediaSource; return mediaSource;
} }
return new AdsMediaSource( return new AdsMediaSource(
......
...@@ -195,8 +195,6 @@ public class EventLogger implements AnalyticsListener { ...@@ -195,8 +195,6 @@ public class EventLogger implements AnalyticsListener {
logd( logd(
"mediaItem [" "mediaItem ["
+ getEventTimeString(eventTime) + getEventTimeString(eventTime)
+ ", "
+ (mediaItem == null ? "null" : "mediaId=" + mediaItem.mediaId)
+ ", reason=" + ", reason="
+ getMediaItemTransitionReasonString(reason) + getMediaItemTransitionReasonString(reason)
+ "]"); + "]");
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment